[coreboot-gerrit] Patch set updated for coreboot: c16ec26 util/sconfig/main.c: Free memory pointed to by `outputc`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Tue Aug 5 08:07:08 CEST 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6487

-gerrit

commit c16ec26a6c7cc79319be73eb291e95fb2d09441d
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Mon Aug 4 08:22:39 2014 +0200

    util/sconfig/main.c: Free memory pointed to by `outputc`
    
    Cppcheck 1.65 reports the error below.
    
    	[main.c:709]: (error) Memory leak: outputc
    
    So free the memory space pointed to by `outputc` to fix the memory
    leak.
    
    Change-Id: I1767cd15380832a93cc79fb515861982e3f1ee94
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 util/sconfig/main.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/util/sconfig/main.c b/util/sconfig/main.c
index 91fb8cb..03d7a03 100644
--- a/util/sconfig/main.c
+++ b/util/sconfig/main.c
@@ -631,6 +631,7 @@ int main(int argc, char** argv) {
 		perror(NULL);
 		exit(1);
 	}
+	free(outputc);
 
 	struct header *h;
 	if (scan_mode == STATIC_MODE) {



More information about the coreboot-gerrit mailing list