[PATCH 05/10] setexpr: Split the core logic into its own function
Tom Rini
trini at konsulko.com
Wed Dec 2 22:22:57 CET 2020
On Sun, Nov 01, 2020 at 02:15:39PM -0700, Simon Glass wrote:
> At present this function always allocates a large amount of stack, and
> selects its own size for buffers. This makes it hard to test the code
> for buffer overflow.
>
> Separate out the inner logic of the substitution so that tests can call
> this directly. This will allow checking that the algorithm does not
> overflow the buffer.
>
> Fix up one of the error lines at the same time, since it should be
> printing nbuf_size, not data_size.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
Applied to u-boot/next, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20201202/e1fdf8eb/attachment.sig>
More information about the U-Boot
mailing list