diff --git a/lib/fields.c b/lib/fields.c
index 9e08adadf..ef8283741 100644
--- a/lib/fields.c
+++ b/lib/fields.c
@@ -43,7 +43,7 @@ int valid_field (const char *field, const char *illegal)
 
 	/* For each character of field, search if it appears in the list
 	 * of illegal characters. */
-	if (illegal && NULL != strpbrk (field, illegal)) {
+	if (illegal && strpbrk(field, illegal)) {
 		return -1;
 	}
 
diff --git a/lib/setupenv.c b/lib/setupenv.c
index b7dcb3fa1..eba81734b 100644
--- a/lib/setupenv.c
+++ b/lib/setupenv.c
@@ -72,7 +72,7 @@ static void read_env_file (const char *filename)
 		val = stpsep(cp, "=");
 		if (val == NULL)
 			continue;
-		if (strpbrk(name, " \t") != NULL)
+		if (strpbrk(name, " \t"))
 			continue;
 #if 0				/* XXX untested, and needs rewrite with fewer goto's :-) */
 /*