Jacob Garber has uploaded this change for review.

View Change

linux_mtd: Fix param memory leak

extract_programmer_param() stores allocated memory in param, so make
sure it is freed at the end of the function.

Change-Id: I363e66b49c1ed4034ac058b94a938c8bb197e048
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: Coverity CID 1403823
---
M linux_mtd.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/47/34847/1
diff --git a/linux_mtd.c b/linux_mtd.c
index ae8bef2..d2df95e 100644
--- a/linux_mtd.c
+++ b/linux_mtd.c
@@ -404,5 +404,6 @@

ret = 0;
linux_mtd_init_exit:
+ free(param);
return ret;
}

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I363e66b49c1ed4034ac058b94a938c8bb197e048
Gerrit-Change-Number: 34847
Gerrit-PatchSet: 1
Gerrit-Owner: Jacob Garber <jgarber1@ualberta.ca>
Gerrit-MessageType: newchange