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

thor: cmd: free memory allocated by strdup()

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

diff --git a/common/cmd_thordown.c b/common/cmd_thordown.c
index a5e6587cae..423fb94556 100644
--- a/common/cmd_thordown.c
+++ b/common/cmd_thordown.c
@@ -31,7 +31,8 @@ int do_thor_down(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 4:
@@ -91,6 +92,11 @@ done:
 	} else
 		lcd_clear();
 #endif
+bad_args:
+	free(usb_controller);
+	free(interface);
+	free(devstring);
+
 	return ret;
 }