mbox series

[0/9] Coverity fixes

Message ID 20180319045420.22046-1-cyril.bur@au1.ibm.com
Headers show
Series Coverity fixes | expand

Message

Cyril Bur March 19, 2018, 4:54 a.m. UTC 
Coverity is back everyone! It found some more stuff.

I the last patch removes FIXMEs. They MAY have been caused by 8/9. I
can't test it nor do I know the code well enough to be sure. Feel to
free to drop the last patch on the floor!

Cyril Bur (9):
  occ-sensors: Remove NULL checks after dereference
  npu2: Fix possible NULL dereference
  npu2-opencapi: Fix memory leak
  libstb/create-container: munmap() signature file address
  fast-reboot: occ: Only delete /ibm,opal/power-mgt nodes if they exist
  hw/imc: Don't dereference possible NULL
  p9_stop_api: Only update SCOM restore entry if there is an entry
  dts: Zero struct to avoid using uninitialised value
  [RFC] dts: Possible FIX for set trip bits

 hw/dts.c                  | 19 ++-----------------
 hw/imc.c                  |  3 ++-
 hw/npu2-opencapi.c        |  2 +-
 hw/npu2.c                 |  9 ++++++---
 hw/occ-sensor.c           |  4 ++--
 hw/occ.c                  | 12 +++++++-----
 libpore/p9_stop_api.C     |  2 +-
 libstb/container.h        |  4 ++--
 libstb/create-container.c | 36 +++++++++++++++++++-----------------
 9 files changed, 42 insertions(+), 49 deletions(-)

Comments

Stewart Smith March 27, 2018, 4:50 a.m. UTC | #1 
Cyril Bur <cyril.bur@au1.ibm.com> writes:
> Coverity is back everyone! It found some more stuff.
>
> I the last patch removes FIXMEs. They MAY have been caused by 8/9. I
> can't test it nor do I know the code well enough to be sure. Feel to
> free to drop the last patch on the floor!
>
> Cyril Bur (9):
>   occ-sensors: Remove NULL checks after dereference
>   npu2: Fix possible NULL dereference
>   npu2-opencapi: Fix memory leak
>   libstb/create-container: munmap() signature file address
>   fast-reboot: occ: Only delete /ibm,opal/power-mgt nodes if they exist
>   hw/imc: Don't dereference possible NULL
>   p9_stop_api: Only update SCOM restore entry if there is an entry
>   dts: Zero struct to avoid using uninitialised value
>   [RFC] dts: Possible FIX for set trip bits

I've picked them all up except for the p9_stop_api one (I'll poke the PM
team for it) and the RFC (because RFC and there should be thought).

Merged to master as of 351b05be3d403cd444db4390a5d9110631240cb6