| commit | 9a4114a15a4a63d059923c54ab08d392ca856c03 | [log] [tgz] |
|---|---|---|
| author | Mathias Krause <mathias.krause@secunet.com> | Wed Feb 08 10:31:42 2012 +0100 |
| committer | Peter Stuge <peter@stuge.se> | Fri Feb 10 15:15:01 2012 +0100 |
| tree | 3e37661dc76634b6d03f00f57512bd542e5d17e2 | |
| parent | bdc8c83614f18ff87e978a0b1d60a27a61ec7c0d [diff] [blame] |
libpayload: fix possible mem leak in get_option_as_string() Change-Id: I7c3adbd1b72be81585bbaabb42532fc4cad57f58 Signed-off-by: Mathias Krause <mathias.krause@secunet.com> Reviewed-on: http://review.coreboot.org/616 Tested-by: build bot (Jenkins) Reviewed-by: Peter Stuge <peter@stuge.se>
diff --git a/payloads/libpayload/drivers/options.c b/payloads/libpayload/drivers/options.c index 10d165e..8161c22 100644 --- a/payloads/libpayload/drivers/options.c +++ b/payloads/libpayload/drivers/options.c
@@ -292,7 +292,7 @@ *dest = strdup((const char*)cmos_enum->text); break; default: /* fail */ - return 1; + ret = 1; } free(raw); return ret;