lib/, src/: Use str[n]cmp(3) instead of explicit byte comparisons

This is simpler to read, IMO.

Signed-off-by: Alejandro Colomar <[email protected]>

Author: alejandro-colomar

lib/utmp.c

@@ -410,12 +410,12 @@ active_sessions_count(const char *name, unsigned long limit)
 		if (USER_PROCESS != ut->ut_type) {
 			continue;
 		}
-		if ('\0' == ut->ut_user[0]) {
+		if (strncmp(ut->ut_user, "", NITEMS(ut->ut_user)) == 0)
 			continue;
-		}
-		if (strncmp (name, ut->ut_user, sizeof (ut->ut_user)) != 0) {
+
+		if (strncmp(ut->ut_user, name, NITEMS(ut->ut_user)) != 0)
 			continue;
-		}
+
 		count++;
 		if (count > limit) {
 			break;

src/logoutd.c

@@ -13,6 +13,7 @@
 #include <fcntl.h>
 #include <signal.h>
 #include <stdio.h>
+#include <string.h>
 #include <sys/stat.h>
 #include <sys/types.h>
 #include <utmpx.h>
@@ -178,9 +179,9 @@ main(int argc, char **argv)
 			if (ut->ut_type != USER_PROCESS) {
 				continue;
 			}
-			if (ut->ut_user[0] == '\0') {
+			if (strncmp(ut->ut_user, "", NITEMS(ut->ut_user)) == 0)
 				continue;
-			}
+
 			if (check_login (ut)) {
 				continue;
 			}