[U-Boot] MPC85xx: Uninitialized variables and other problems

Wolfgang Denk wd at denx.de
Thu Nov 6 12:54:51 CET 2014


Hello,

cppcheck reports:

[arch/powerpc/cpu/mpc85xx/cmd_errata.c:62]: (error) Uninitialized variable: x108

[board/freescale/common/cds_pci_ft.c:36]: (error) Possible null pointer dereference: map
[board/freescale/common/cds_pci_ft.c:52]: (error) Uninitialized variable: len

[board/freescale/t4qds/eth.c:289]: (error) Undefined behavior: Variable 'buf' is used as parameter and destination in s[n]printf().
[board/freescale/t4qds/eth.c:298]: (error) Undefined behavior: Variable 'buf' is used as parameter and destination in s[n]printf().
[board/freescale/t4qds/eth.c:307]: (error) Undefined behavior: Variable 'buf' is used as parameter and destination in s[n]printf().
[board/freescale/t4qds/eth.c:316]: (error) Undefined behavior: Variable 'buf' is used as parameter and destination in s[n]printf().

[drivers/ddr/fsl/ctrl_regs.c:1659]: (error) Uninitialized variable: wr
[drivers/ddr/fsl/ctrl_regs.c:2257]: (error) Uninitialized variable: ea
[drivers/ddr/fsl/ctrl_regs.c:2256]: (error) Uninitialized variable: sa
[drivers/ddr/fsl/ctrl_regs.c:1985]: (error) Uninitialized variable: zqcs_init

[drivers/mmc/fsl_esdhc_spl.c:95]: (error) Dereferencing 'tmp_buf' after it is deallocated / released
[drivers/mmc/fsl_esdhc_spl.c:98]: (error) Memory pointed to by 'tmp_buf' is freed twice.
[drivers/mmc/fsl_esdhc_spl.c:101]: (error) Dereferencing 'tmp_buf' after it is deallocated / released
[drivers/mmc/fsl_esdhc_spl.c:104]: (error) Memory pointed to by 'tmp_buf' is freed twice.



Can you please have a look and fix these?

Thanks in advance.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A person who is more than casually interested in computers should  be
well  schooled in machine language, since it is a fundamental part of
a computer.                                           -- Donald Knuth


More information about the U-Boot mailing list