Attention is currently required from: eshankelkar@galorithm.com.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/71791
to look at the new patch set (#3).
Change subject: sfdp.c : Initializing hbuf to avoid warnings generated by scan-build ......................................................................
sfdp.c : Initializing hbuf to avoid warnings generated by scan-build
scan-build uses the clang analyzer which is giving warnings about garbage/undefined values in hbuf assigned to member of hdrs[0] if hbuf is not initialized.
Though the path of the control flow shown by it in its html output cannot occur in reality(since it assumes that (nph+1)*8 is <= 0 i.e nph<=-1 but later assumes 0 <= nph ) hence its a false positive. Still initializing all bytes of hbuf to 0 leads to the two warnings for sfdp.c (one for hbuf, one for tbuf) to go away.
Signed-off-by : Eshan Kelkar eshankelkar@galorithm.com
Change-Id: I6815e246b4fd225d1837cae6e7d2aa0236b48b1b --- M sfdp.c 1 file changed, 28 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/91/71791/3