summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorChalapathi V <chalapathi.v@linux.ibm.com>2024-08-07 22:28:04 +0200
committerNicholas Piggin <npiggin@gmail.com>2024-11-04 09:09:15 +1000
commit031324472eee57bce9bd4a0231aa9b137494d8a1 (patch)
treee1ab7c7e71afad3d8ca1100cd8bfc7b2537d7991
parent3feabc18ad4d4bdc178a205b986353a54dbfcf20 (diff)
downloadfocaccia-qemu-031324472eee57bce9bd4a0231aa9b137494d8a1.tar.gz
focaccia-qemu-031324472eee57bce9bd4a0231aa9b137494d8a1.zip
hw/ssi/pnv_spi: Fixes Coverity CID 1558831
In this commit the following coverity scan defect has been fixed
CID 1558831:  Resource leaks  (RESOURCE_LEAK)
  Variable "rsp_payload" going out of scope leaks the storage it
  points to.

Cc: qemu-stable@nongnu.org
Fixes: Coverity CID 1558831
Signed-off-by: Chalapathi V <chalapathi.v@linux.ibm.com>
Fixes: b4cb930e40 ("hw/ssi: Extend SPI model")
[PMD: Rebased on previous commit (returning earlier)]
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
-rw-r--r--hw/ssi/pnv_spi.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/ssi/pnv_spi.c b/hw/ssi/pnv_spi.c
index 72bcc00de7..c21b2ebb3c 100644
--- a/hw/ssi/pnv_spi.c
+++ b/hw/ssi/pnv_spi.c
@@ -239,6 +239,7 @@ static void transfer(PnvSpi *s, PnvXferBuffer *payload)
         }
     }
     spi_response(s, s->N1_bits, rsp_payload);
+    pnv_spi_xfer_buffer_free(rsp_payload);
 }
 
 static inline uint8_t get_seq_index(PnvSpi *s)