[PATCH 05/10] setexpr: Split the core logic into its own function

Tom Rini
2020-12-02T21:22:57Z

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@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>