utmp: call prepare_utmp() even if utent is NULL

update_utmp() should also return 0 when success.

Fixes: 1f368e1c1838de9d476a36897d7c53394569de08 ("utmp: update
`update_utmp()")
Resolves: https://github.com/shadow-maint/shadow/issues/805

Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>

diff --git a/lib/utmp.c b/lib/utmp.c
index 9b173327ac7ad90e9c77bd2c3997402eb19a4f7d..e0ae48a0830448e1ec768fe4774aa6c8c49605c5 100644 (file)
--- a/lib/utmp.c
+++ b/lib/utmp.c
@@ -370,17 +370,16 @@ int update_utmp (const char *user,
        struct utmp *utent, *ut;
 
        utent = get_current_utmp ();
-       if (utent == NULL) {
-               return -1;
-       }
-
        ut = prepare_utmp  (user, tty, host, utent);
 
        (void) setutmp  (ut);   /* make entry in the utmp & wtmp files */
-       free (utent);
+
+       if (utent != NULL) {
+               free (utent);
+       }
        free (ut);
 
-       return 1;
+       return 0;
 }
 
 void record_failure(const char *failent_user,