diff options
| author |   millert <millert@openbsd.org> | 2002-10-16 19:45:41 +0000 |
|---|---|---|
| committer | millert <millert@openbsd.org> | 2002-10-16 19:45:41 +0000 |
| commit | 08d1e6e62791d518df51cc3778e3b3cb6118f7b5 (patch) | |
| tree | f0814ed2b205e81603497e926db7f3d2ec11de08 /lib/libc | |
| parent | sprinkle const; mostly from NetBSD (diff) | |
| download | wireguard-openbsd-08d1e6e62791d518df51cc3778e3b3cb6118f7b5.tar.xz wireguard-openbsd-08d1e6e62791d518df51cc3778e3b3cb6118f7b5.zip | |
Replace "if (value != NULL && value == as->FOO)" with just
"if (value == as->FOO)" -- NULL needs no special handling.
Pointed out by Paul Borman.
Diffstat (limited to 'lib/libc')
| -rw-r--r-- | lib/libc/gen/auth_subr.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/libc/gen/auth_subr.c b/lib/libc/gen/auth_subr.c index db13471bb61..6aab2bfcd63 100644 --- a/lib/libc/gen/auth_subr.c +++ b/lib/libc/gen/auth_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: auth_subr.c,v 1.16 2002/10/15 20:17:11 millert Exp $ */ +/* $OpenBSD: auth_subr.c,v 1.17 2002/10/16 19:45:41 millert Exp $ */ /*- * Copyright (c) 1995,1996,1997 Berkeley Software Design, Inc. @@ -428,7 +428,7 @@ auth_setitem(auth_session_t *as, auth_item_t item, char *value) return (0); case AUTHV_CHALLENGE: - if (value != NULL && value == as->challenge) + if (value == as->challenge) return (0); if (value != NULL && (value = strdup(value)) == NULL) return (-1); @@ -438,7 +438,7 @@ auth_setitem(auth_session_t *as, auth_item_t item, char *value) return (0); case AUTHV_CLASS: - if (value != NULL && value == as->class) + if (value == as->class) return (0); if (value != NULL && (value = strdup(value)) == NULL) return (-1); @@ -448,7 +448,7 @@ auth_setitem(auth_session_t *as, auth_item_t item, char *value) return (0); case AUTHV_NAME: - if (value != NULL && value == as->name) + if (value == as->name) return (0); if (value != NULL && (value = strdup(value)) == NULL) return (-1); @@ -458,7 +458,7 @@ auth_setitem(auth_session_t *as, auth_item_t item, char *value) return (0); case AUTHV_SERVICE: - if (value != NULL && value == as->service) + if (value == as->service) return (0); if (value == NULL || strcmp(value, defservice) == 0) value = defservice; @@ -470,7 +470,7 @@ auth_setitem(auth_session_t *as, auth_item_t item, char *value) return (0); case AUTHV_STYLE: - if (value != NULL && value == as->style) + if (value == as->style) return (0); if (value == NULL || strchr(value, '/') != NULL || (value = strdup(value)) == NULL) |