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

Simon Glass sjg at chromium.org
Sun Apr 28 21:38:27 UTC 2019


On Fri, 26 Apr 2019 at 10:39, Heinrich Schuchardt <xypron.glpk at 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 at 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 at chromium.org>


More information about the U-Boot mailing list