Patrick Georgi has uploaded this change for review.

View Change

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, &current);
 	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;
 	}
 

To view, visit change 40650. To unsubscribe, or for help writing mail filters, visit settings.