d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

The new test(1) did not use access() correctly. I don't know why, since

Browse Source 
supposedly it's ksh-derived, and it's not broken in pdksh. I've added
a test for test running as root: if testing for -x, the file must be
mode & 0111 to get "success", rather than just existant.

Reviewed by:	chris
This commit is contained in:
Brian Feldman 1999-08-18 00:18:52 +00:00
parent 944494356b
commit eb5e5558e9
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
bin/test/test.c
View File

@ -12,7 +12,7 @@
#ifndef lint
static const char rcsid[] =
"$Id: test.c,v 1.22 1999/08/14 05:38:04 chris Exp $";
"$Id: test.c,v 1.23 1999/08/16 09:44:09 sheldonh Exp $";
#endif /* not lint */
#include <sys/types.h>
@ -328,7 +328,12 @@ filstat(nm, mode)
case FILWR:
return access(nm, W_OK) == 0;
case FILEX:
return access(nm, X_OK) == 0;
if (access(nm, X_OK) == 0) {
if (getuid() == 0 && (s.st_mode & 0111) == 0)
return 0;
return 1;
}
return 1;
case FILEXIST:
return access(nm, F_OK) == 0;
case FILREG: