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

fixed a bug that caused directory specs in a Cmnd_Alias to fail if the

Browse Source 
last entry in the spec failed (ie: it was only looking at the last entry).
CLeaned things up by adding the cmndcmp() function--all neat & tidy
This commit is contained in:
Todd C. Miller
1994-08-13 01:22:10 +00:00
parent c3be7c93c4
commit 971a69bb26
1 changed files with 26 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
parse.c
View File

@@ -81,6 +81,7 @@ LINK tmp_ptr, reset_ptr, save_ptr, list_ptr[NUM_LISTS];
* Prototypes
*/
static int hostcmp __P((char *));
static int cmndcmp __P((char *, char *));
/*
@@ -351,19 +352,9 @@ int cmnd_type_ok()
list_ptr[CMND_LIST] = tmp_ptr -> next;
while (next_type == TYPE3) {
/*
* Check to see if a directory is being permitted
* Match cmnd to the data (directory or file)
*/
if (list_ptr[CMND_LIST]->
data[strlen(list_ptr[CMND_LIST]->data)-1] == '/' ) {
/* we have a directory spec */
if (strncmp(list_ptr[CMND_LIST]->data, cmnd,
strlen(list_ptr[CMND_LIST]->data)) == 0)
return(MATCH);
else
return(NO_MATCH);
}
if (strcmp(list_ptr[CMND_LIST] -> data, cmnd) == 0) {
if (cmndcmp(cmnd, list_ptr[CMND_LIST] -> data) == 0) {
if (list_ptr[USER_LIST] -> op == '!') {
list_ptr[CMND_LIST] = save_ptr;
return (QUIT_NOW);
@@ -535,3 +526,26 @@ static int hostcmp(target)
return(strcmp(target, host));
}
}
/*
* this routine is called from cmnd_type_ok() and tries to match a cmnd
* to a data entry from the sudoers file.
*/
static int cmndcmp(cmnd, data)
char *cmnd; /* command the user is attempting */
char *data; /* data we are checking against */
{
int len = strlen(data);
/*
* If the data is a directory, match based on len,
* otherwise do a normal strcmp(3)
*/
if (*(data + len - 1) == '/')
return(strncmp(data, cmnd, len));
else
return(strcmp(data, cmnd));
}