[Skiboot] [PATCH 0/8] Random static analysis fixes

Stewart Smith stewart at linux.ibm.com
Tue Dec 11 17:24:07 AEDT 2018


Stewart Smith <stewart at linux.ibm.com> writes:
> 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

Series merged to master as of e35f1cb63c1bedd3710a3a0fa7bea9f68910fb86

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list