[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