[flashrom-gerrit] Change in flashrom[master]: cli_classic: fix memory leak

[Anastasia Klimchuk (Code Review)]

Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/64351 )

Change subject: cli_classic: fix memory leak
......................................................................

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 
Change-Id: I8520e302e9d63ed1215c5d9beb90a93fb52a91fe
Reviewed-on: https://review.coreboot.org/c/flashrom/+/64351
Tested-by: build bot (Jenkins) 
Reviewed-by: Thomas Heijligen 
Reviewed-by: Anastasia Klimchuk 
Reviewed-by: Nikolai Artemiev 
---
M cli_classic.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Thomas Heijligen: Looks good to me, approved
  Nikolai Artemiev: Looks good to me, but someone else must approve
  Anastasia Klimchuk: Looks good to me, approved

diff --git a/cli_classic.c b/cli_classic.c
index 9543009..83ac538 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -1100,6 +1100,7 @@
    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;