From 549dd239c37162dac062363c390a5a480c03d088 Mon Sep 17 00:00:00 2001 From: Jacob Nevins Date: Sat, 1 Jan 2005 14:06:12 +0000 Subject: [PATCH] Remove rogue sfree()s inside new wildcard action functions -- were causing segfaults in failure cases. [originally from svn r5057] --- psftp.c | 3 --- 1 file changed, 3 deletions(-) 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; }