isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

It is now sudo_efree() not efree().

Browse Source 
Don't try to free a pointer to garbage on error.
This commit is contained in:
Todd C. Miller
2014-07-30 09:46:48 -06:00
parent 080d0e650f
commit 7a988d489c
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/sudo_edit.c
View File

@@ -88,7 +88,7 @@ sudo_edit(struct command_details *command_details)
struct command_details saved_command_details;
ssize_t nread, nwritten;
const char *tmpdir;
char *cp, *suff, **nargv, **ap, **files = NULL;
char *cp, *suff, **nargv = NULL, **ap, **files = NULL;
char buf[BUFSIZ];
int rc, i, j, ac, ofd, tfd, nargc, rval, tmplen;
int editor_argc = 0, nfiles = 0;
@@ -333,8 +333,8 @@ sudo_edit(struct command_details *command_details)
}
close(ofd);
}
efree(tf);
efree(nargv);
sudo_efree(tf);
sudo_efree(nargv);
debug_return_int(rval);
cleanup:
@@ -345,8 +345,8 @@ cleanup:
unlink(tf[i].tfile);
}
}
efree(tf);
efree(nargv);
sudo_efree(tf);
sudo_efree(nargv);
debug_return_int(1);
}