Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/40650 )
Change subject: raiden_debug_spi: Fix memleak ......................................................................
raiden_debug_spi: Fix memleak
Change-Id: I2701a8fbae63657edc9cc258666cc951f92b1115 Found-by: Coverity Scan #1420204 Signed-off-by: Patrick Georgi pgeorgi@google.com --- M raiden_debug_spi.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/50/40650/1
diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c index 173e355..a0d6578 100644 --- a/raiden_debug_spi.c +++ b/raiden_debug_spi.c @@ -480,12 +480,14 @@ ret = LIBUSB(libusb_init(NULL)); if (ret != 0) { msg_perr("Raiden: libusb_init failed\n"); + free(serial); return ret; }
ret = usb_device_find(&match, ¤t); if (ret != 0) { msg_perr("Raiden: Failed to find devices\n"); + free(serial); return ret; }
@@ -546,6 +548,7 @@
if (!device || !found) { msg_perr("Raiden: No usable device found.\n"); + free(serial); return 1; }