diff options
| author | Tom Rini <trini@konsulko.com> | 2023-04-06 09:58:40 -0400 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2023-04-25 15:31:28 -0400 |
| commit | e97c89cc15b90c72dcf6a7a943fb52743a1f70fa (patch) | |
| tree | d2e1be260cb9b08cdd56e9b956643d7c1c87d76f | |
| parent | ac44c6cf957a91cea5271334c2d6e6cd867cb5d6 (diff) | |
| download | u-boot-e97c89cc15b90c72dcf6a7a943fb52743a1f70fa.zip u-boot-e97c89cc15b90c72dcf6a7a943fb52743a1f70fa.tar.gz u-boot-e97c89cc15b90c72dcf6a7a943fb52743a1f70fa.tar.bz2 | |
qemu: dfu: Correct memset call in set_dfu_alt_info
When building with clang, we see:
board/emulation/common/qemu_dfu.c:51:24: warning: 'memset' call operates on objects of type 'char' while the size is based on a different type 'char *' [-Wsizeof-pointer-memaccess]
As we're calling memset with the length set to the size of the pointer
and not the size of the buffer. Correct this with a call of the size of
the buffer itself.
Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Acked-by: Sughosh Ganu <sughosh.ganu@linaro.org>
| -rw-r--r-- | board/emulation/common/qemu_dfu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/emulation/common/qemu_dfu.c b/board/emulation/common/qemu_dfu.c index 332d659..7e7d84f 100644 --- a/board/emulation/common/qemu_dfu.c +++ b/board/emulation/common/qemu_dfu.c @@ -48,7 +48,7 @@ void set_dfu_alt_info(char *interface, char *devstr) env_get("dfu_alt_info")) return; - memset(buf, 0, sizeof(buf)); + memset(buf, 0, DFU_ALT_BUF_LEN); /* * Currently dfu_alt_info is needed on Qemu ARM64 for |