Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
raiden_debug_spi: Fix memleak

Change-Id: Ib9d99fefda812d20265db47be353c844f8b77129
Found-by: Coverity Scan #1420204
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/40969
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
---
M raiden_debug_spi.c
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c
index 368476f..2127f69 100644
--- a/raiden_debug_spi.c
+++ b/raiden_debug_spi.c
@@ -480,8 +480,10 @@
int ret;

int request_enable = get_target();
- if (request_enable < 0)
+ if (request_enable < 0) {
+ free(serial);
return 1;
+ }

usb_match_init(&match);


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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib9d99fefda812d20265db47be353c844f8b77129
Gerrit-Change-Number: 40969
Gerrit-PatchSet: 3
Gerrit-Owner: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged