Nico Huber submitted this change.
print: Fix vendor string memory leak
Freeing this string won't really matter in the incredible case that we
run out of memory, but it keeps Coverity happy.
Change-Id: I962d2f2227850473b70272bc48b3fc0a0fb11342
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: Coverity CID 1403822
Reviewed-on: https://review.coreboot.org/c/flashrom/+/34849
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: David Hendricks <david.hendricks@gmail.com>
---
M print.c
1 file changed, 1 insertion(+), 0 deletions(-)
diff --git a/print.c b/print.c
index 2901c50..91e5ae5 100644
--- a/print.c
+++ b/print.c
@@ -184,6 +184,7 @@
dev = malloc(strlen(chip->name) + 1);
if (dev == NULL) {
msg_gerr("Out of memory!\n");
+ free(ven);
return 1;
}
strcpy(dev, chip->name);
To view, visit change 34849. To unsubscribe, or for help writing mail filters, visit settings.