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 48c12a4..71a8f27 100644 (file)
--- 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;
     }