[Skiboot] [PATCH 0/8] Random static analysis fixes
Stewart Smith
stewart at linux.ibm.com
Thu Nov 29 15:28:25 AEDT 2018
Peeking into a dusty static analysis tool I have running, there's a few
things we could easily clean up to get fewer random noisy warnings.
Probably worthwhile as this tool only spits out ~90 for skiboot, some of
which look possibly legit.
No functional change here, but there was some grumbling at static
analysis tools.
Stewart Smith (8):
core/cpu.c: avoid container_of(NULL) in next_cpu()
Change ifdef around dump_fdt() to shut up static analysis
mem_region.c: Move ifdef for MEM_POISON to shut up static analysis
opal_trace_entry: Move ifdef around to shut up static analysis
opal_sync_host_reboot: clarify when we return OPAL_BUSY_EVENT
vpd: Force static analysis to not think about NULL term strings
hdata/iohub.c: remove condition that was always true
hw/bt.c: Move some debug ifdef to make static analysis happy
core/cpu.c | 10 +++++-----
core/fdt.c | 4 ++--
core/mem_region.c | 8 ++++++--
core/opal.c | 16 ++++++++--------
core/vpd.c | 2 +-
hdata/iohub.c | 3 +--
hw/bt.c | 6 ++++--
7 files changed, 27 insertions(+), 22 deletions(-)
--
2.19.2
More information about the Skiboot
mailing list