From: jacob <jacob@cda61777-01e9-0310-a592-d414129be87e>
Date: Sat, 1 Jan 2005 14:06:12 +0000 (+0000)
Subject: Remove rogue sfree()s inside new wildcard action functions -- were causing
X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/commitdiff_plain/3bbbd50d44728c2bc72f5abee5d73020d592bb0e?ds=sidebyside

Remove rogue sfree()s inside new wildcard action functions -- were causing
segfaults in failure cases.


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

diff --git a/psftp.c b/psftp.c
index 48c12a4e..71a8f274 100644
--- a/psftp.c
+++ b/psftp.c
@@ -1457,7 +1457,6 @@ static int sftp_action_rm(void *vctx, char *fname)
 
     if (!result) {
 	printf("rm %s: %s\n", fname, fxp_error());
-	sfree(fname);
 	return 0;
     }
 
@@ -1624,7 +1623,6 @@ static int sftp_action_chmod(void *vctx, char *fname)
     if (!result || !(attrs.flags & SSH_FILEXFER_ATTR_PERMISSIONS)) {
 	printf("get attrs for %s: %s\n", fname,
 	       result ? "file permissions not provided" : fxp_error());
-	sfree(fname);
 	return 0;
     }
 
@@ -1644,7 +1642,6 @@ static int sftp_action_chmod(void *vctx, char *fname)
 
     if (!result) {
 	printf("set attrs for %s: %s\n", fname, fxp_error());
-	sfree(fname);
 	return 0;
     }