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 a885fd814755af8df8757cfe6a7120a8996b5bac..5fd25ea12e52fe51d07dd118eb690b9c35d1fb02 100644 (file)
--- 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,7 +95,12 @@ done:
                run_command("reset", 0);
 
        g_dnl_clear_detach();
-
+bad_args:
+       if (argc == 1) {
+               free(usb_controller);
+               free(interface);
+               free(devstring);
+       }
        return ret;
 }