handle_ssh_variant: plug memory leak

The split_cmdline() function either assigns an allocated array to the argv
parameter or NULL. In the first case, we have to free() it, in the latter
it does no harm.

Reported via Coverity.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

connect.c

@@ -716,8 +716,10 @@ static int handle_ssh_variant(const char *ssh_command, int is_cmdline,
 			 * any longer.
 			 */
 			free(ssh_argv);
-		} else
+		} else {
+			free(ssh_argv);
 			return 0;
+		}
 	}
 
 	if (!strcasecmp(variant, "plink") ||