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

Avoid calling fclose(NULL) if the sudoers file is not secure and

Browse Source 
restore_perms() fails.  Coverity CID 104090.
This commit is contained in:
Todd C. Miller
2016-05-05 15:01:22 -06:00
parent 5725acd1c4
commit 64142f9da2
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
plugins/sudoers/sudoers.c
View File

@@ -939,8 +939,10 @@ open_sudoers(const char *sudoers, bool doedit, bool *keepopen)
if (!restore_perms()) { if (!restore_perms()) {
/* unable to change back to root */ /* unable to change back to root */
fclose(fp); if (fp != NULL) {
fp = NULL; fclose(fp);
fp = NULL;
}
} }
debug_return_ptr(fp); debug_return_ptr(fp);