Attention is currently required from: Felix Singer, Thomas Heijligen, Angel Pons, Anastasia Klimchuk.
Hello build bot (Jenkins), Miklós Márton, Thomas Heijligen, Anastasia Klimchuk,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/67700
to look at the new patch set (#2).
Change subject: stlinkv3_spi: fix uninitialized pointer ......................................................................
stlinkv3_spi: fix uninitialized pointer
`stlinkv3_handle` declared without an initial value. The variable is initialized in a branch which can only be accessed if `devs_stlinkv3_spi[0].vendor_id != 0`. Otherwise, the variable contains a garbage value. We can consider this case as a false positive because `devs_stlinkv3_spi` holds as a minimum one device entry (otherwise we wouldn't need a driver).
This issue was found by setting compiler flag `-Og`, which optimizes debugging experience, and running scan-build. So, we have to fix it to allow the compiler to use the flag and remove the warning from the scan-build list.
Change-Id: Ibaf25f67186724d9045ade849026782c3eac4952 Signed-off-by: Alexander Goncharov chat@joursoir.net --- M stlinkv3_spi.c 1 file changed, 24 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/00/67700/2