don't leak argument list; bz3404, reported by Balu Gajjala
authordjm <djm@openbsd.org>
Sun, 20 Mar 2022 08:52:17 +0000 (08:52 +0000)
committerdjm <djm@openbsd.org>
Sun, 20 Mar 2022 08:52:17 +0000 (08:52 +0000)
ok dtucker@

usr.bin/ssh/scp.c

index d2f6252..fc5d99e 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: scp.c,v 1.246 2022/02/23 19:01:00 deraadt Exp $ */
+/* $OpenBSD: scp.c,v 1.247 2022/03/20 08:52:17 djm Exp $ */
 /*
  * scp - secure remote copy.  This is basically patched BSD rcp which
  * uses ssh to do the data transfer (instead of using rcmd).
@@ -935,7 +935,7 @@ do_sftp_connect(char *host, char *user, int port, char *sftp_direct,
                        return NULL;
 
        } else {
-               args.list = NULL;
+               freeargs(&args);
                addargs(&args, "sftp-server");
                if (do_cmd(sftp_direct, host, NULL, -1, 0, "sftp",
                    reminp, remoutp, pidp) < 0)