Skip to content

Commit

Permalink
Merge pull request #26 from johnthesecond/fix_issue_19
Browse files Browse the repository at this point in the history
See issue 19. This should fix the issue if uncommented again.
  • Loading branch information
luke-goddard authored Aug 17, 2020
2 parents e70feb4 + 6e223b3 commit 0cd8148
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/scans/permissions_scan.c
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,8 @@ static void check_global_write(All_Results *ar, File_Info *fi)
// add_issue(LOW, AUDIT, fi->location, ar, issue_buf, "ENUMY failed to stat the parent directory");
// return;
// }
// struct passwd *data = getpwuid(stats.st_uid);
// // John: getpwuid is not re-entrant. Using getpwuid_r should fix this issue.
// struct passwd *data = getpwuid_r(stats.st_uid);
// if (data == NULL)
// {
// log_error_errno_loc(ar, "Failed to stat directory", parent_buf, errno);
Expand Down Expand Up @@ -319,4 +320,4 @@ static void check_readable_sen_file(All_Results *ar, char *location)
/* Check if current non root users can write to fstab */
else if (access(location, R_OK) && (getuid() != 0))
add_issue(HIGH, CTF, location, ar, "Current user can read from very important file", "");
}
}

0 comments on commit 0cd8148

Please sign in to comment.