From: simon <simon@cda61777-01e9-0310-a592-d414129be87e>
Date: Wed, 12 Sep 2001 20:11:48 +0000 (+0000)
Subject: psftp and pscp should disable all forwarding (ports, X, agent).
X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/commitdiff_plain/d27b4a18b02db3bab2bb9f096183b093c7ff27ef

psftp and pscp should disable all forwarding (ports, X, agent).
Partly because that's a good idea _anyway_, and partly because it
seems to be causing trouble. (Specifically, their pathetic attempt
to emulate plink's proper select handling seems to get confused when
the back end tries to open a local listening socket.)


git-svn-id: svn://svn.tartarus.org/sgt/putty@1264 cda61777-01e9-0310-a592-d414129be87e
---

diff --git a/psftp.c b/psftp.c
index 37f39ff3..137320b8 100644
--- a/psftp.c
+++ b/psftp.c
@@ -1710,6 +1710,15 @@ int main(int argc, char *argv[])
     /* SFTP uses SSH2 by default always */
     cfg.sshprot = 2;
 
+    /*
+     * Disable scary things which shouldn't be enabled for simple
+     * things like SCP and SFTP: agent forwarding, port forwarding,
+     * X forwarding.
+     */
+    cfg.x11_forward = 0;
+    cfg.agentfwd = 0;
+    cfg.portfwd[0] = cfg.portfwd[1] = '\0';
+
     /* Set up subsystem name. */
     strcpy(cfg.remote_cmd, "sftp");
     cfg.ssh_subsys = TRUE;
diff --git a/scp.c b/scp.c
index 7a8cebde..f4c8d794 100644
--- a/scp.c
+++ b/scp.c
@@ -604,6 +604,15 @@ static void do_cmd(char *host, char *user, char *cmd)
 	cfg.port = portnumber;
 
     /*
+     * Disable scary things which shouldn't be enabled for simple
+     * things like SCP and SFTP: agent forwarding, port forwarding,
+     * X forwarding.
+     */
+    cfg.x11_forward = 0;
+    cfg.agentfwd = 0;
+    cfg.portfwd[0] = cfg.portfwd[1] = '\0';
+
+    /*
      * Attempt to start the SFTP subsystem as a first choice,
      * falling back to the provided scp command if that fails.
      */