* src/login_nopam.c: Do not use the YES and NO macros. Use the

booleans true and false instead. Change the prototypes of
	list_match(), user_match(), from_match(), and string_match()
	accordingly. Also use booleans internally.
	* src/login_nopam.c: Add brackets and parenthesis.
	* src/login_nopam.c: Avoid implicit conversion of pointers /
	integers / chars to booleans.
	* src/login_nopam.c: Avoid assignments in comparisons.
This commit is contained in:
nekral-guest 2008-06-09 18:35:32 +00:00
parent 4e0d734598
commit 46466a8fcc
2 changed files with 104 additions and 77 deletions

View File

@ -1,3 +1,14 @@
2008-06-09 Nicolas François <nicolas.francois@centraliens.net>
* src/login_nopam.c: Do not use the YES and NO macros. Use the
booleans true and false instead. Change the prototypes of
list_match(), user_match(), from_match(), and string_match()
accordingly. Also use booleans internally.
* src/login_nopam.c: Add brackets and parenthesis.
* src/login_nopam.c: Avoid implicit conversion of pointers /
integers / chars to booleans.
* src/login_nopam.c: Avoid assignments in comparisons.
2008-06-09 Nicolas François <nicolas.francois@centraliens.net> 2008-06-09 Nicolas François <nicolas.francois@centraliens.net>
* src/newgrp.c: Use a bool for is_newgrp, notfound, needspasswd, * src/newgrp.c: Use a bool for is_newgrp, notfound, needspasswd,

View File

@ -66,14 +66,10 @@
static char fs[] = ":"; /* field separator */ static char fs[] = ":"; /* field separator */
static char sep[] = ", \t"; /* list-element separator */ static char sep[] = ", \t"; /* list-element separator */
/* Constants to be used in assignments only, not in comparisons... */ static bool list_match (char *list, const char *item, bool (*match_fn) (const char *, const char *));
#define YES 1 static bool user_match (const char *tok, const char *string);
#define NO 0 static bool from_match (const char *tok, const char *string);
static bool string_match (const char *tok, const char *string);
static int list_match (char *list, const char *item, int (*match_fn) (const char *, const char *));
static int user_match (const char *tok, const char *string);
static int from_match (const char *tok, const char *string);
static int string_match (const char *tok, const char *string);
static const char *resolve_hostname (const char *string); static const char *resolve_hostname (const char *string);
/* login_access - match username/group and host/tty with access control file */ /* login_access - match username/group and host/tty with access control file */
@ -84,7 +80,7 @@ int login_access (const char *user, const char *from)
char *perm; /* becomes permission field */ char *perm; /* becomes permission field */
char *users; /* becomes list of login names */ char *users; /* becomes list of login names */
char *froms; /* becomes list of terminals or hosts */ char *froms; /* becomes list of terminals or hosts */
int match = NO; bool match = false;
int end; int end;
int lineno = 0; /* for diagnostics */ int lineno = 0; /* for diagnostics */
@ -95,26 +91,31 @@ int login_access (const char *user, const char *from)
* mandatory. The first field should be a "+" or "-" character. A * mandatory. The first field should be a "+" or "-" character. A
* non-existing table means no access control. * non-existing table means no access control.
*/ */
if ((fp = fopen (TABLE, "r"))) { fp = fopen (TABLE, "r");
while (!match && fgets (line, sizeof (line), fp)) { if (NULL != fp) {
while (!match && (fgets (line, sizeof (line), fp) == line)) {
lineno++; lineno++;
if (line[end = strlen (line) - 1] != '\n') { end = (int) strlen (line) - 1;
if (line[end] != '\n') {
SYSLOG ((LOG_ERR, SYSLOG ((LOG_ERR,
"%s: line %d: missing newline or line too long", "%s: line %d: missing newline or line too long",
TABLE, lineno)); TABLE, lineno));
continue; continue;
} }
if (line[0] == '#') if (line[0] == '#') {
continue; /* comment line */ continue; /* comment line */
while (end > 0 && isspace (line[end - 1])) }
while (end > 0 && isspace (line[end - 1])) {
end--; end--;
line[end] = 0; /* strip trailing whitespace */ }
if (line[0] == 0) /* skip blank lines */ line[end] = '\0'; /* strip trailing whitespace */
if (line[0] == '\0') { /* skip blank lines */
continue; continue;
if (!(perm = strtok (line, fs)) }
|| !(users = strtok ((char *) 0, fs)) if ( ((perm = strtok (line, fs)) == NULL)
|| !(froms = strtok ((char *) 0, fs)) || ((users = strtok ((char *) 0, fs)) == NULL)
|| strtok ((char *) 0, fs)) { || ((froms = strtok ((char *) 0, fs)) == NULL)
|| (strtok ((char *) 0, fs) != NULL)) {
SYSLOG ((LOG_ERR, SYSLOG ((LOG_ERR,
"%s: line %d: bad field count", "%s: line %d: bad field count",
TABLE, lineno)); TABLE, lineno));
@ -133,14 +134,14 @@ int login_access (const char *user, const char *from)
} else if (errno != ENOENT) { } else if (errno != ENOENT) {
SYSLOG ((LOG_ERR, "cannot open %s: %m", TABLE)); SYSLOG ((LOG_ERR, "cannot open %s: %m", TABLE));
} }
return (match == 0 || (line[0] == '+')); return (!match || (line[0] == '+'))?1:0;
} }
/* list_match - match an item against a list of tokens with exceptions */ /* list_match - match an item against a list of tokens with exceptions */
static int list_match (char *list, const char *item, int (*match_fn) (const char *, const char*)) static bool list_match (char *list, const char *item, bool (*match_fn) (const char *, const char*))
{ {
char *tok; char *tok;
int match = NO; bool match = false;
/* /*
* Process tokens one at a time. We have exhausted all possible matches * Process tokens one at a time. We have exhausted all possible matches
@ -148,22 +149,26 @@ static int list_match (char *list, const char *item, int (*match_fn) (const char
* a match, look for an "EXCEPT" list and recurse to determine whether * a match, look for an "EXCEPT" list and recurse to determine whether
* the match is affected by any exceptions. * the match is affected by any exceptions.
*/ */
for (tok = strtok (list, sep); tok != 0; tok = strtok ((char *) 0, sep)) { for (tok = strtok (list, sep); tok != NULL; tok = strtok ((char *) 0, sep)) {
if (strcasecmp (tok, "EXCEPT") == 0) /* EXCEPT: give up */ if (strcasecmp (tok, "EXCEPT") == 0) { /* EXCEPT: give up */
break; break;
if ((match = (*match_fn) (tok, item))) /* YES */ }
match = (*match_fn) (tok, item);
if (match) {
break; break;
} }
}
/* Process exceptions to matches. */ /* Process exceptions to matches. */
if (match != NO) { if (match) {
while ((tok = strtok ((char *) 0, sep)) while ( ((tok = strtok ((char *) 0, sep)) != NULL)
&& strcasecmp (tok, "EXCEPT")) && (strcasecmp (tok, "EXCEPT") != 0))
/* VOID */ ; /* VOID */ ;
if (tok == 0 || list_match ((char *) 0, item, match_fn) == NO) if (tok == 0 || !list_match ((char *) 0, item, match_fn)) {
return (match); return (match);
} }
return (NO); }
return false;
} }
/* myhostname - figure out local machine name */ /* myhostname - figure out local machine name */
@ -171,33 +176,33 @@ static char *myhostname (void)
{ {
static char name[MAXHOSTNAMELEN + 1] = ""; static char name[MAXHOSTNAMELEN + 1] = "";
if (name[0] == 0) { if (name[0] == '\0') {
gethostname (name, sizeof (name)); gethostname (name, sizeof (name));
name[MAXHOSTNAMELEN] = 0; name[MAXHOSTNAMELEN] = '\0';
} }
return (name); return (name);
} }
#if HAVE_INNETGR #if HAVE_INNETGR
/* netgroup_match - match group against machine or user */ /* netgroup_match - match group against machine or user */
static int static bool
netgroup_match (const char *group, const char *machine, const char *user) netgroup_match (const char *group, const char *machine, const char *user)
{ {
static char *mydomain = 0; static char *mydomain = (char *)0;
if (mydomain == 0) { if (mydomain == (char *)0) {
static char domain[MAXHOSTNAMELEN + 1]; static char domain[MAXHOSTNAMELEN + 1];
getdomainname (domain, MAXHOSTNAMELEN); getdomainname (domain, MAXHOSTNAMELEN);
mydomain = domain; mydomain = domain;
} }
return innetgr (group, machine, user, mydomain); return (innetgr (group, machine, user, mydomain) != 0);
} }
#endif #endif
/* user_match - match a username against one token */ /* user_match - match a username against one token */
static int user_match (const char *tok, const char *string) static bool user_match (const char *tok, const char *string)
{ {
struct group *group; struct group *group;
@ -209,11 +214,12 @@ static int user_match (const char *tok, const char *string)
/* /*
* If a token has the magic value "ALL" the match always succeeds. * If a token has the magic value "ALL" the match always succeeds.
* Otherwise, return YES if the token fully matches the username, or if * Otherwise, return true if the token fully matches the username, or if
* the token is a group that contains the username. * the token is a group that contains the username.
*/ */
if ((at = strchr (tok + 1, '@')) != 0) { /* split user@host pattern */ at = strchr (tok + 1, '@');
*at = 0; if (NULL != at) { /* split user@host pattern */
*at = '\0';
return ( user_match (tok, string) return ( user_match (tok, string)
&& from_match (at + 1, myhostname ())); && from_match (at + 1, myhostname ()));
#if HAVE_INNETGR #if HAVE_INNETGR
@ -221,12 +227,14 @@ static int user_match (const char *tok, const char *string)
return (netgroup_match (tok + 1, (char *) 0, string)); return (netgroup_match (tok + 1, (char *) 0, string));
#endif #endif
} else if (string_match (tok, string)) { /* ALL or exact match */ } else if (string_match (tok, string)) { /* ALL or exact match */
return (YES); return true;
/* local, no need for xgetgrnam */ /* local, no need for xgetgrnam */
} else if ((group = getgrnam (tok))) { /* try group membership */ } else if ((group = getgrnam (tok)) != NULL) { /* try group membership */
for (i = 0; group->gr_mem[i]; i++) for (i = 0; NULL != group->gr_mem[i]; i++) {
if (strcasecmp (string, group->gr_mem[i]) == 0) if (strcasecmp (string, group->gr_mem[i]) == 0) {
return (YES); return true;
}
}
#ifdef PRIMARY_GROUP_MATCH #ifdef PRIMARY_GROUP_MATCH
/* /*
* If the string is an user whose initial GID matches the token, * If the string is an user whose initial GID matches the token,
@ -237,12 +245,15 @@ static int user_match (const char *tok, const char *string)
* getpwnam() doesn't have some nasty side effects. --marekm * getpwnam() doesn't have some nasty side effects. --marekm
*/ */
/* local, no need for xgetpwnam */ /* local, no need for xgetpwnam */
if ((userinf = getpwnam (string))) userinf = getpwnam (string);
if (userinf->pw_gid == group->gr_gid) if (NULL != userinf) {
return (YES); if (userinf->pw_gid == group->gr_gid) {
return true;
}
}
#endif #endif
} }
return (NO); return false;
} }
static const char *resolve_hostname (const char *string) static const char *resolve_hostname (const char *string)
@ -254,8 +265,9 @@ static const char *resolve_hostname (const char *string)
struct hostent *hp; struct hostent *hp;
hp = gethostbyname (string); hp = gethostbyname (string);
if (hp) if (NULL != hp) {
return inet_ntoa (*((struct in_addr *) *(hp->h_addr_list))); return inet_ntoa (*((struct in_addr *) *(hp->h_addr_list)));
}
SYSLOG ((LOG_ERR, "%s - unknown host", string)); SYSLOG ((LOG_ERR, "%s - unknown host", string));
return string; return string;
@ -263,17 +275,17 @@ static const char *resolve_hostname (const char *string)
/* from_match - match a host or tty against a list of tokens */ /* from_match - match a host or tty against a list of tokens */
static int from_match (const char *tok, const char *string) static bool from_match (const char *tok, const char *string)
{ {
int tok_len; size_t tok_len;
int str_len; size_t str_len;
/* /*
* If a token has the magic value "ALL" the match always succeeds. Return * If a token has the magic value "ALL" the match always succeeds. Return
* YES if the token fully matches the string. If the token is a domain * true if the token fully matches the string. If the token is a domain
* name, return YES if it matches the last fields of the string. If the * name, return true if it matches the last fields of the string. If the
* token has the magic value "LOCAL", return YES if the string does not * token has the magic value "LOCAL", return true if the string does not
* contain a "." character. If the token is a network number, return YES * contain a "." character. If the token is a network number, return true
* if it matches the head of the string. * if it matches the head of the string.
*/ */
#if HAVE_INNETGR #if HAVE_INNETGR
@ -282,35 +294,39 @@ static int from_match (const char *tok, const char *string)
} else } else
#endif #endif
if (string_match (tok, string)) { /* ALL or exact match */ if (string_match (tok, string)) { /* ALL or exact match */
return (YES); return true;
} else if (tok[0] == '.') { /* domain: match last fields */ } else if (tok[0] == '.') { /* domain: match last fields */
if ((str_len = strlen (string)) > (tok_len = strlen (tok)) str_len = strlen (string);
&& strcasecmp (tok, string + str_len - tok_len) == 0) tok_len = strlen (tok);
return (YES); if ( (str_len > tok_len)
} else if (strcasecmp (tok, "LOCAL") == 0) { /* local: no dots */ && (strcasecmp (tok, string + str_len - tok_len) == 0)) {
if (strchr (string, '.') == 0) return true;
return (YES);
} else if (tok[(tok_len = strlen (tok)) - 1] == '.' /* network */
&& strncmp (tok, resolve_hostname (string), tok_len) == 0) {
return (YES);
} }
return (NO); } else if (strcasecmp (tok, "LOCAL") == 0) { /* local: no dots */
if (strchr (string, '.') == NULL) {
return true;
}
} else if ( (tok[(tok_len = strlen (tok)) - 1] == '.') /* network */
&& (strncmp (tok, resolve_hostname (string), tok_len) == 0)) {
return true;
}
return false;
} }
/* string_match - match a string against one token */ /* string_match - match a string against one token */
static int string_match (const char *tok, const char *string) static bool string_match (const char *tok, const char *string)
{ {
/* /*
* If the token has the magic value "ALL" the match always succeeds. * If the token has the magic value "ALL" the match always succeeds.
* Otherwise, return YES if the token fully matches the string. * Otherwise, return true if the token fully matches the string.
*/ */
if (strcasecmp (tok, "ALL") == 0) { /* all: always matches */ if (strcasecmp (tok, "ALL") == 0) { /* all: always matches */
return (YES); return true;
} else if (strcasecmp (tok, string) == 0) { /* try exact match */ } else if (strcasecmp (tok, string) == 0) { /* try exact match */
return (YES); return true;
} }
return (NO); return false;
} }
#else /* !USE_PAM */ #else /* !USE_PAM */