Re: [U-Boot] [PATCH 1/1] lib/display_options: avoid illegal memory access

On Fri, 26 Apr 2019 at 10:39, Heinrich Schuchardt xypron.glpk@gmx.de wrote:

display_options_get_banner_priv() overwrites bytes before the start of the buffer if the buffer size is less then 3. This case occurs in the Sandbox when executing the `ut_print` command.

Correctly handle small buffer sizes. Adjust the print unit test to catch when bytes before the buffer are overwritten.

Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de

I will take the patch via the u-boot-efi repository.

lib/display_options.c | 4 +++- test/print_ut.c | 20 ++++++++++++-------- 2 files changed, 15 insertions(+), 9 deletions(-)

Reviewed-by: Simon Glass sjg@chromium.org