login-utils: cleanup strlen() and fgets() usage

The strlen() returns size_t, not int. The fgets() returns
pointer, not int.

Signed-off-by: James Youngman <jay@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

diff --git a/login-utils/islocal.c b/login-utils/islocal.c
index 1ee84089fe04d2e2d949ca42c0480d285740e54a..297698041f44de7446079b5c32e8cef96c571765 100644 (file)
--- a/login-utils/islocal.c
+++ b/login-utils/islocal.c
@@ -32,7 +32,7 @@ is_local(char *user)
        FILE *fd;
        char line[MAX_LENGTH];
        int local = 0;
-       int len;
+       size_t len;
 
         if(!(fd = fopen(_PATH_PASSWD, "r"))) {
                 fprintf(stderr,_("Can't read %s, exiting."),_PATH_PASSWD);
@@ -40,7 +40,7 @@ is_local(char *user)
         }
 
        len = strlen(user);
-        while(fgets(line, MAX_LENGTH, fd) > 0) {
+        while(fgets(line, MAX_LENGTH, fd)) {
                 if(!strncmp(line, user, len) && line[len] == ':') {
                        local = 1;
                        break;