Many places in the code NULL is used in integer context, where

plain 0 should be used.  This happens to work because we #define
NULL to 0, but is stylistically wrong and can cause problems
for people trying to port bits of code to other environments.

PR:		2752
Submitted by:	Arne Henrik Juul <arnej@imf.unit.no>

3 files changed, 4 insertions, 4 deletions

diff --git a/lib/libc/db/btree/bt_seq.c b/lib/libc/db/btree/bt_seq.c
index 303b481..76e6275 100644
--- a/lib/libc/db/btree/bt_seq.c
+++ b/lib/libc/db/btree/bt_seq.c
@@ -358,7 +358,7 @@ __bt_first(t, key, erval, exactp)
 	 * page) and return it.
 	 */
 	if ((ep = __bt_search(t, key, exactp)) == NULL)
-		return (NULL);
+		return (0);
 	if (*exactp) {
 		if (F_ISSET(t, B_NODUPS)) {
 			*erval = *ep;
diff --git a/lib/libc/gen/getcap.c b/lib/libc/gen/getcap.c
index 93d1952..a7de288 100644
--- a/lib/libc/gen/getcap.c
+++ b/lib/libc/gen/getcap.c
@@ -735,7 +735,7 @@ cgetnext(bp, db_array)
 			}
 		}
 		rp = buf;
-		for(cp = nbuf; *cp != NULL; cp++)
+		for(cp = nbuf; *cp != '\0'; cp++)
 			if (*cp == '|' || *cp == ':')
 				break;
 			else
diff --git a/lib/libc/gen/getpwent.c b/lib/libc/gen/getpwent.c
index 9a19a15..dfe5d34 100644
--- a/lib/libc/gen/getpwent.c
+++ b/lib/libc/gen/getpwent.c
@@ -424,7 +424,7 @@ grpagain:
 				latch++;
 			}
 again:
-			if (getnetgrent(&host, &user, &domain) == NULL) {
+			if (getnetgrent(&host, &user, &domain) == 0) {
 				if ((gr = getgrnam(grp+2)) != NULL)
 					goto grpagain;
 				latch = 0;
@@ -448,7 +448,7 @@ again:
 		if (grp[1] == '@') {
 			setnetgrent(grp+2);
 			rv = 0;
-			while(getnetgrent(&host, &user, &domain) != NULL) {
+			while(getnetgrent(&host, &user, &domain) != 0) {
 				store(user);
 				rv++;
 			}