From 2d9d88ed6c96f03724bc77fdeee04b9f6b8b68de Mon Sep 17 00:00:00 2001
From: Alexander Goncharov <chat@joursoir.net>
Date: Sat, 14 May 2022 23:42:33 +0300
Subject: cli_classic: fix memory leak

If parameter --wp-region is used and wp_region is allocated,
free the last one at the exit of the application.

Found-by: scan-build, clang v13.0.1
Signed-off-by: Alexander Goncharov <chat@joursoir.net>
Change-Id: I8520e302e9d63ed1215c5d9beb90a93fb52a91fe
Reviewed-on: https://review.coreboot.org/c/flashrom/+/64351
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Thomas Heijligen <src@posteo.de>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Nikolai Artemiev <nartemiev@google.com>
---
 cli_classic.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cli_classic.c b/cli_classic.c
index 95430098..83ac5383 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -1100,6 +1100,7 @@ out:
 	free(referencefile);
 	free(layoutfile);
 	free(pparam);
+	free(wp_region);
 	/* clean up global variables */
 	free((char *)chip_to_probe); /* Silence! Freeing is not modifying contents. */
 	chip_to_probe = NULL;
-- 
cgit v1.2.3