[Skiboot] [PATCH] hdat: logically dead code, assert instead

Stewart Smith stewart at linux.vnet.ibm.com
Wed Sep 20 18:44:50 AEST 2017


Stewart Smith <stewart at linux.vnet.ibm.com> writes:
> Since we already assert that base != NULL, the if()
> condition on hdif can never be NULL. Instead, we should
> just assert (if even that) if it is.
>
> Found by Coverity
>
> Signed-off-by: Stewart Smith <stewart at linux.vnet.ibm.com>
> ---
>  hdata/spira.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Merged to master as of 8599e8ad69052e5dfcc2625dab1211302fb45f9a


-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list