diff options
author | Jacob Garber <jgarber1@ualberta.ca> | 2019-07-02 10:50:10 -0600 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2019-07-07 20:25:12 +0000 |
commit | f2ba2d9421744eb31e4001f448ad02593dd82883 (patch) | |
tree | f3b92594abf002dc982a0006fc3f8210483be124 /src/lib/memmove.c | |
parent | 3c2305f162d24cf07fa6079bd20fd2839e5d672c (diff) |
arch/x86: Use ssize_t to store length
size_t is the wrong type to store the return value of
acpi_device_path_fill(), since any negative error values will be
converted to a very large unsigned integer and potentially cause
buffer overflow.
Change-Id: Ia8ed62ecfac8eaa18a61545bd203b3c7a7cd9ca5
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: Coverity CID 1402095
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33962
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/lib/memmove.c')
0 files changed, 0 insertions, 0 deletions