From: Przemyslaw Marczak <p.marczak@samsung.com>
Date: Fri, 28 Nov 2014 12:56:20 +0000 (+0100)
Subject: dfu: cmd: free memory allocated by strdup()
X-Git-Tag: submit/tizen/20141210.090340~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4ff6ab392a9986857a8532803b4e185374dd0708;p=platform%2Fkernel%2Fu-boot.git

dfu: cmd: free memory allocated by strdup()

Change-Id: I69f49736af033314f9f4681eeb97fb11a0499b8b
Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
---

diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c
index b102b7dcc4..40c3ccd83e 100644
--- a/common/cmd_dfu.c
+++ b/common/cmd_dfu.c
@@ -32,7 +32,8 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
 		if (!usb_controller || !interface || !devstring) {
 			puts("DFU: default device environment is not set.\n");
-			return CMD_RET_USAGE;
+			ret = CMD_RET_USAGE;
+			goto bad_args;
 		}
 		break;
 	case 5:
@@ -94,6 +95,10 @@ done:
 		run_command("reset", 0);
 
 	dfu_clear_detach();
+bad_args:
+	free(usb_controller);
+	free(interface);
+	free(devstring);
 
 	return ret;
 }