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 a5e6587cae8b5b3617131245a0c02ee119cbf939..423fb94556dadda100619e5afbf637ef3f8dde71 100644 (file)
--- 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;
 }