Re: sandbox_spl build warning

On 3/11/23 02:38, Simon Glass wrote:

Hi Marek,

I see this in CI for -next:

Building current source for 1 boards (0 threads, 64 jobs per thread) sandbox: w+ sandbox_spl

• 1159 | fdt_test_header_get(uts, fdt, "magic", fdt_magic(fdt));

•  |         ^
  

+test/cmd/fdt.c:1159:9: note: referencing argument 2 of type ‘char *’ +test/cmd/fdt.c:1115:12: note: in a call to function ‘fdt_test_header_get’

• 1115 | static int fdt_test_header_get(struct unit_test_state *uts,

char fdt[4096],

•  |            ^
  

• 1160 | fdt_test_header_get(uts, fdt, "totalsize", fdt_totalsize(fdt));

here:

https://source.denx.de/u-boot/custodians/u-boot-dm/-/jobs/591918

Could you please take a look?

[PATCH] test: cmd: fdt: Drop unused fdt_test_header_get() fdt parameter

should fix that. It's the Wstringop-overflow which, for whatever reason, I am not getting on my machine.

Please give it a try.