Message ID | 1417612031-16552-1-git-send-email-angelo@sysam.it |
---|---|
State | Deferred |
Delegated to: | Tom Rini |
Headers | show |
Hi Angelo, > Compiler still complains about headersz possible uninitialized, > even after last change (switch used). > > CROSS_COMPILE=/opt/toolchains/m68k/gcc-4.6.3-nolibc/m68k-linux/ > bin/m68k-linux- ./MAKEALL -a m68k > boards.cfg is up to date. Nothing to do. > Building M52277EVB board... > text data bss dec hex filename > 118493 11756 4096 134345 20cc9 ./u-boot > tools/kwbimage.c: In function ‘kwbimage_set_header’: > tools/kwbimage.c:803:8: warning: ‘headersz’ may be used > uninitialized in this function [-Wmaybe-uninitialized] > memcpy(ptr, image, headersz); > ^ > > Signed-off-by: Angelo Dureghello <angelo@sysam.it> > --- > tools/kwbimage.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/kwbimage.c b/tools/kwbimage.c > index c50f2e2..2c302e5 100644 > --- a/tools/kwbimage.c > +++ b/tools/kwbimage.c > @@ -728,7 +728,7 @@ static void kwbimage_set_header(void *ptr, struct > stat *sbuf, int ifd, FILE *fcfg; > void *image = NULL; > int version; > - size_t headersz; > + size_t headersz = 0; > uint32_t checksum; > int ret; > int size; You aren't the first person who attempted to fix this. Please read the following thread: http://patchwork.ozlabs.org/patch/412968/ Albert has already promised to look into this issue. Best regards, Lukasz Majewski
Dear Lukasz, > Hi Angelo, > You aren't the first person who attempted to fix this. i read the history of the fixes done on this file before posting, and seen the issue was solved using a "switch / case" approach. I was testing yesterday the kernel.org x86 toolchain for m68k, (m68k/gcc-4.6.3-nolibc) and found this warning emitted building the master branch,so i posted the patch since is possible the warning is thrown only with this m68k toolchain. So i don't want absolutely overlap ongoing jobs, if they are already in chargeto anyone simply ignore the patch. Best Regards, Angelo Dureghello
diff --git a/tools/kwbimage.c b/tools/kwbimage.c index c50f2e2..2c302e5 100644 --- a/tools/kwbimage.c +++ b/tools/kwbimage.c @@ -728,7 +728,7 @@ static void kwbimage_set_header(void *ptr, struct stat *sbuf, int ifd, FILE *fcfg; void *image = NULL; int version; - size_t headersz; + size_t headersz = 0; uint32_t checksum; int ret; int size;
Compiler still complains about headersz possible uninitialized, even after last change (switch used). CROSS_COMPILE=/opt/toolchains/m68k/gcc-4.6.3-nolibc/m68k-linux/ bin/m68k-linux- ./MAKEALL -a m68k boards.cfg is up to date. Nothing to do. Building M52277EVB board... text data bss dec hex filename 118493 11756 4096 134345 20cc9 ./u-boot tools/kwbimage.c: In function ‘kwbimage_set_header’: tools/kwbimage.c:803:8: warning: ‘headersz’ may be used uninitialized in this function [-Wmaybe-uninitialized] memcpy(ptr, image, headersz); ^ Signed-off-by: Angelo Dureghello <angelo@sysam.it> --- tools/kwbimage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)