]> andersk Git - openssh.git/blobdiff - loginrec.c
Hopefully things did not get mixed around too much. It compiles under
[openssh.git] / loginrec.c
index cfa35acc10e3e01ace4c61c862f613b9a8f4bc00..b664a9cfec86ef329223376359a546be38b7ecce 100644 (file)
 
 /**
  ** TODO:
- **   homegrown ttyslot()q
+ **   homegrown ttyslot()
  **   test, test, test
  **
  ** Platform status:
  ** ----------------
  **
  ** Known good:
- **   Linux (Redhat 6.2, need more variants)
+ **   Linux (Redhat 6.2, Debian)
+ **   Solaris
  **   HP-UX 10.20 (gcc only)
+ **   IRIX
+ **   NeXT - M68k/HPPA/Sparc (4.2/3.3)
  **
  ** Testing required: Please send reports!
- **   Solaris
- **   IRIX
  **   NetBSD
  **   HP-UX 11
  **   AIX
  **
  ** Platforms with known problems:
- **   NeXT
+ **   Some variants of Slackware Linux
  **
  **/
 
 #include "includes.h"
 
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <netinet/in.h>
-#include <unistd.h>
-#include <fcntl.h>
-#include <stdlib.h>
-#include <string.h>
-#include <stdio.h>
-#include <errno.h>
-#ifdef HAVE_PWD_H
-#  include <pwd.h>
-#endif
-#ifdef HAVE_SYS_TIME_H
-#  include <sys/time.h>
-#else
-#  include <time.h>
-#endif
-#if HAVE_UTMP_H
-# include <utmp.h>
-#endif
-#if HAVE_UTMPX_H
-# include <utmpx.h>
-#endif
-#if HAVE_LASTLOG_H
-# include <lastlog.h>
-#endif
-
 #include "ssh.h"
 #include "xmalloc.h"
 #include "loginrec.h"
+#include "log.h"
+#include "atomicio.h"
 
 RCSID("$Id$");
 
+#ifdef HAVE_UTIL_H
+#  include <util.h>
+#endif
+
+#ifdef HAVE_LIBUTIL_H
+#   include <libutil.h>
+#endif
 
 /**
  ** prototypes for helper functions in this file
@@ -216,12 +199,22 @@ int lastlog_get_entry(struct logininfo *li);
 int wtmp_get_entry(struct logininfo *li);
 int wtmpx_get_entry(struct logininfo *li);
 
+/* pick the shortest string */
+#define MIN_SIZEOF(s1,s2) ( sizeof(s1) < sizeof(s2) ? sizeof(s1) : sizeof(s2) )
 
 /**
  ** platform-independent login functions
  **/
 
-/* Record a login */
+/* login_login(struct logininfo *)     -Record a login
+ * 
+ * Call with a pointer to a struct logininfo initialised with
+ * login_init_entry() or login_alloc_entry()
+ *
+ * Returns:
+ *  >0 if successful
+ *  0  on failure (will use OpenSSH's logging facilities for diagnostics)
+ */
 int
 login_login (struct logininfo *li)
 {
@@ -230,7 +223,14 @@ login_login (struct logininfo *li)
 }
 
 
-/* Record a logout */
+/* login_logout(struct logininfo *)     - Record a logout
+ *
+ * Call as with login_login()
+ *
+ * Returns:
+ *  >0 if successful
+ *  0  on failure (will use OpenSSH's logging facilities for diagnostics)
+ */
 int
 login_logout(struct logininfo *li)
 {
@@ -238,44 +238,97 @@ login_logout(struct logininfo *li)
        return login_write(li);
 }
 
-
-/* Retrieve the last login time for a user (or fake on from wtmp/wtmpx) */
+/* login_get_lastlog_time(int)           - Retrieve the last login time
+ *
+ * Retrieve the last login time for the given uid. Will try to use the
+ * system lastlog facilities if they are available, but will fall back
+ * to looking in wtmp/wtmpx if necessary
+ *
+ * Returns:
+ *   0 on failure, or if user has never logged in
+ *   Time in seconds from the epoch if successful
+ *
+ * Useful preprocessor symbols:
+ *   DISABLE_LASTLOG: If set, *never* even try to retrieve lastlog
+ *                    info
+ *   USE_LASTLOG: If set, indicates the presence of system lastlog
+ *                facilities. If this and DISABLE_LASTLOG are not set,
+ *                try to retrieve lastlog information from wtmp/wtmpx.
+ */
 unsigned int
 login_get_lastlog_time(const int uid)
 {
        struct logininfo li;
 
-       login_get_lastlog(&li, uid);
-       return li.tv_sec;
+       if (login_get_lastlog(&li, uid))
+               return li.tv_sec;
+       else
+               return 0;
 }
 
-/* Retrieve a lastlog entry (or fake one from wtmp/wtmpx) */
+/* login_get_lastlog(struct logininfo *, int)   - Retrieve a lastlog entry
+ *
+ * Retrieve a logininfo structure populated (only partially) with
+ * information from the system lastlog data, or from wtmp/wtmpx if no
+ * system lastlog information exists.
+ *
+ * Note this routine must be given a pre-allocated logininfo.
+ *
+ * Returns:
+ *  >0: A pointer to your struct logininfo if successful
+ *  0  on failure (will use OpenSSH's logging facilities for diagnostics)
+ *
+ */
 struct logininfo *
 login_get_lastlog(struct logininfo *li, const int uid)
 {
-       memset(li, '\0', sizeof(struct logininfo));
+       struct passwd *pw;
+
+       memset(li, '\0', sizeof(*li));
        li->uid = uid;
+
+       /* 
+        * If we don't have a 'real' lastlog, we need the username to
+        * reliably search wtmp(x) for the last login (see
+        * wtmp_get_entry().) 
+        */
+       pw = getpwuid(uid);
+       if (pw == NULL)
+               fatal("login_get_lastlog: Cannot find account for uid %i", uid);
+               
+       /* No MIN_SIZEOF here - we absolutely *must not* truncate the
+         * username */
+       strlcpy(li->username, pw->pw_name, sizeof(li->username));
+
        if (getlast_entry(li))
                return li;
        else
-               return 0;
+               return NULL;
 }
 
 
-/* login_alloc_entry()    - allocate and initialise a logininfo */
+/* login_alloc_entry(int, char*, char*, char*)    - Allocate and initialise
+ *                                                  a logininfo structure 
+ * 
+ * This function creates a new struct logininfo, a data structure
+ * meant to carry the information required to portably record login info.
+ *
+ * Returns a pointer to a newly created struct logininfo. If memory
+ * allocation fails, the program halts.
+ */
 struct
 logininfo *login_alloc_entry(int pid, const char *username,
                             const char *hostname, const char *line)
 {
        struct logininfo *newli;
 
-       newli = (struct logininfo *) xmalloc (sizeof(struct logininfo));
+       newli = (struct logininfo *) xmalloc (sizeof(*newli));
        (void)login_init_entry(newli, pid, username, hostname, line);
        return newli;
 }
 
 
-/* login_free_entry()    - free struct memory (trivial) */
+/* login_free_entry(struct logininfo *)    - free struct memory */
 void
 login_free_entry(struct logininfo *li)
 {
@@ -283,43 +336,59 @@ login_free_entry(struct logininfo *li)
 }
 
 
-/* login_init_entry()   - initialise a struct logininfo */
+/* login_init_entry(struct logininfo *, int, char*, char*, char*)
+ *                                        - initialise a struct logininfo
+ * 
+ * Populates a new struct logininfo, a data structure meant to carry
+ * the information required to portably record login info.
+ *
+ * Returns: 1
+ */
 int
 login_init_entry(struct logininfo *li, int pid, const char *username, 
                 const char *hostname, const char *line)
 {
-       /* zero the structure */
-       memset(li, 0, sizeof(struct logininfo));
+       struct passwd *pw;
+       
+       memset(li, 0, sizeof(*li));
   
        li->pid = pid;
+
        /* set the line information */
        if (line)
                line_fullname(li->line, line, sizeof(li->line));
 
-       if (username)
+       if (username) {
                strlcpy(li->username, username, sizeof(li->username));
+               pw = getpwnam(li->username);
+               if (pw == NULL)
+                       fatal("login_init_entry: Cannot find user \"%s\"", li->username);
+               li->uid = pw->pw_uid;
+       }
+
        if (hostname)
                strlcpy(li->hostname, hostname, sizeof(li->hostname));
+
        return 1;
 }
 
-
+/* login_set_current_time(struct logininfo *)    - set the current time
+ *
+ * Set the current time in a logininfo structure. This function is
+ * meant to eliminate the need to deal with system dependencies for
+ * time handling.
+ */
 void
 login_set_current_time(struct logininfo *li)
 {
-#ifdef HAVE_SYS_TIME_H
        struct timeval tv;
 
        gettimeofday(&tv, NULL);
-       li->tv_sec = tv.tv_sec ; li->tv_usec = tv.tv_usec;
-#else
-       time_t tm = time(0);
-
-       li->tv_sec = tm; li->tv_usec = 0;
-#endif
+       
+       li->tv_sec = tv.tv_sec;
+       li->tv_usec = tv.tv_usec;
 }
 
-
 /* copy a sockaddr_* into our logininfo */
 void
 login_set_addr(struct logininfo *li, const struct sockaddr *sa,
@@ -339,14 +408,16 @@ login_set_addr(struct logininfo *li, const struct sockaddr *sa,
  ** login_write: Call low-level recording functions based on autoconf
  ** results
  **/
-
 int
 login_write (struct logininfo *li)
 {
+#ifndef HAVE_CYGWIN
        if ((int)geteuid() != 0) {
          log("Attempt to write login records by non-root user (aborting)");
          return 1;
        }
+#endif
+
        /* set the timestamp */
        login_set_current_time(li);
 #ifdef USE_LOGIN
@@ -372,7 +443,6 @@ login_write (struct logininfo *li)
        return 0;
 }
 
-
 /**
  ** getlast_entry: Call low-level functions to retrieve the last login
  **                time.
@@ -383,38 +453,30 @@ int
 getlast_entry(struct logininfo *li)
 {
 #ifdef USE_LASTLOG
-       if (lastlog_get_entry(li))
-               return 1;
-       else
-               return 0;
-#else
-       /* !USE_LASTLOG */
+       return(lastlog_get_entry(li));
+#else /* !USE_LASTLOG */
 
+#ifdef DISABLE_LASTLOG
+       /* On some systems we shouldn't even try to obtain last login 
+        * time, e.g. AIX */
+       return 0;
+# else /* DISABLE_LASTLOG */
        /* Try to retrieve the last login time from wtmp */
 #  if defined(USE_WTMP) && (defined(HAVE_TIME_IN_UTMP) || defined(HAVE_TV_IN_UTMP))
        /* retrieve last login time from utmp */
-       if (wtmp_get_entry(li))
-               return 1;
-       else
-               return 0;
-#  else
-
+       return (wtmp_get_entry(li));
+#  else /* defined(USE_WTMP) && (defined(HAVE_TIME_IN_UTMP) || defined(HAVE_TV_IN_UTMP)) */
        /* If wtmp isn't available, try wtmpx */
-
-#    if defined(USE_WTMPX) && (defined(HAVE_TIME_IN_UTMPX) || defined(HAVE_TV_IN_UTMPX))
+#   if defined(USE_WTMPX) && (defined(HAVE_TIME_IN_UTMPX) || defined(HAVE_TV_IN_UTMPX))
        /* retrieve last login time from utmpx */
-       if (wtmpx_get_entry(li))
-               return 1;
-       else
-               return 0;
-#    else
-
+       return (wtmpx_get_entry(li));
+#   else
        /* Give up: No means of retrieving last login time */
        return 0;
-#    endif
-#  endif
-#endif
-/* USE_LASTLOG */
+#   endif /* USE_WTMPX && (HAVE_TIME_IN_UTMPX || HAVE_TV_IN_UTMPX) */
+#  endif /* USE_WTMP && (HAVE_TIME_IN_UTMP || HAVE_TV_IN_UTMP) */
+# endif /* DISABLE_LASTLOG */ 
+#endif /* USE_LASTLOG */
 }
 
 
@@ -442,29 +504,27 @@ char *
 line_fullname(char *dst, const char *src, int dstsize)
 {
        memset(dst, '\0', dstsize);
-       if ((strncmp(src, "/dev/", 5) == 0) || (dstsize < (strlen(src) + 5)))
+       if ((strncmp(src, "/dev/", 5) == 0) || (dstsize < (strlen(src) + 5))) {
                strlcpy(dst, src, dstsize);
-       else {
-               strlcpy(dst, "/dev/", 5);
+       else {
+               strlcpy(dst, "/dev/", dstsize);
                strlcat(dst, src, dstsize);
        }
        return dst;
 }
 
-
 /* line_stripname(): strip the leading '/dev' if it exists, return dst */
 char *
 line_stripname(char *dst, const char *src, int dstsize)
 {
        memset(dst, '\0', dstsize);
        if (strncmp(src, "/dev/", 5) == 0)
-               strlcpy(dst, &src[5], dstsize);
+               strlcpy(dst, src + 5, dstsize);
        else
                strlcpy(dst, src, dstsize);
        return dst;
 }
 
-  
 /* line_abbrevname(): Return the abbreviated (usually four-character)
  * form of the line (Just use the last <dstsize> characters of the
  * full name.)
@@ -472,14 +532,34 @@ line_stripname(char *dst, const char *src, int dstsize)
  * NOTE: use strncpy because we do NOT necessarily want zero
  * termination */
 char *
-line_abbrevname(char *dst, const char *src, int dstsize) {
+line_abbrevname(char *dst, const char *src, int dstsize) 
+{
+       size_t len;
+       
        memset(dst, '\0', dstsize);
-       src += (strlen(src) - dstsize);
-       strncpy(dst, src, dstsize); /* note: _don't_ change this to strlcpy */
+       
+       /* Always skip prefix if present */
+#ifdef sgi
+       if (strncmp(src, "/dev/tty", 8) == 0)
+               src += 8;
+#else
+       if (strncmp(src, "/dev/", 5) == 0)
+               src += 5;
+#endif
+               
+       len = strlen(src);
+
+       if (len > 0) {
+               if (((int)len - dstsize) > 0)
+                       src +=  ((int)len - dstsize);
+
+               /* note: _don't_ change this to strlcpy */
+               strncpy(dst, src, (size_t)dstsize); 
+       }
+       
        return dst;
 }
 
-
 /**
  ** utmp utility functions
  **
@@ -493,28 +573,30 @@ line_abbrevname(char *dst, const char *src, int dstsize) {
 void
 set_utmp_time(struct logininfo *li, struct utmp *ut)
 {
-#ifdef HAVE_TV_IN_UTMP
+# ifdef HAVE_TV_IN_UTMP
        ut->ut_tv.tv_sec = li->tv_sec;
        ut->ut_tv.tv_usec = li->tv_usec;
-#else
+# else
 #  ifdef HAVE_TIME_IN_UTMP
        ut->ut_time = li->tv_sec;
 #  endif
-#endif
+# endif
 }
 
-
 void
 construct_utmp(struct logininfo *li,
                    struct utmp *ut)
 {
-       memset(ut, '\0', sizeof(struct utmp));
-#ifdef HAVE_ID_IN_UTMP
+       memset(ut, '\0', sizeof(*ut));
+
+       /* First fill out fields used for both logins and logouts */
+
+# ifdef HAVE_ID_IN_UTMP
        line_abbrevname(ut->ut_id, li->line, sizeof(ut->ut_id));
-#endif
+# endif
 
-#ifdef HAVE_TYPE_IN_UTMP
-       /* This is done here to keep utmp constants out of login.h */
+# ifdef HAVE_TYPE_IN_UTMP
+       /* This is done here to keep utmp constants out of struct logininfo */
        switch (li->type) {
        case LTYPE_LOGIN:
                ut->ut_type = USER_PROCESS;
@@ -523,28 +605,36 @@ construct_utmp(struct logininfo *li,
                ut->ut_type = DEAD_PROCESS;
                break;
        }
-#endif
+# endif
+       set_utmp_time(li, ut);
 
-#ifdef HAVE_PID_IN_UTMP
-       ut->ut_pid = li->pid;
-#endif
        line_stripname(ut->ut_line, li->line, sizeof(ut->ut_line));
-       strlcpy(ut->ut_name, li->username, sizeof(ut->ut_name));
-       set_utmp_time(li, ut);
-#ifdef HAVE_HOST_IN_UTMP
-       strlcpy(ut->ut_host, li->hostname, sizeof(ut->ut_host));
-#endif
-#ifdef HAVE_ADDR_IN_UTMP
+
+# ifdef HAVE_PID_IN_UTMP
+       ut->ut_pid = li->pid;
+# endif
+
+       /* If we're logging out, leave all other fields blank */
+       if (li->type == LTYPE_LOGOUT)
+         return;
+
+       /*
+        * These fields are only used when logging in, and are blank
+        * for logouts. 
+        */
+
+       /* Use strncpy because we don't necessarily want null termination */
+       strncpy(ut->ut_name, li->username, MIN_SIZEOF(ut->ut_name, li->username));
+# ifdef HAVE_HOST_IN_UTMP
+       strncpy(ut->ut_host, li->hostname, MIN_SIZEOF(ut->ut_host, li->hostname));
+# endif
+# ifdef HAVE_ADDR_IN_UTMP
        /* this is just a 32-bit IP address */
        if (li->hostaddr.sa.sa_family == AF_INET)
                ut->ut_addr = li->hostaddr.sa_in.sin_addr.s_addr;
-#endif 
+# endif 
 }
-
-#endif
-/* USE_UTMP || USE_WTMP || USE_LOGIN */
-
-
+#endif /* USE_UTMP || USE_WTMP || USE_LOGIN */
 
 /**
  ** utmpx utility functions
@@ -554,27 +644,27 @@ construct_utmp(struct logininfo *li,
  **/
 
 #if defined(USE_UTMPX) || defined (USE_WTMPX)
-
 /* build the utmpx structure */
 void
 set_utmpx_time(struct logininfo *li, struct utmpx *utx)
 {
-#ifdef HAVE_TV_IN_UTMPX
+# ifdef HAVE_TV_IN_UTMPX
        utx->ut_tv.tv_sec = li->tv_sec;
        utx->ut_tv.tv_usec = li->tv_usec;
-#else
+# else /* HAVE_TV_IN_UTMPX */
 #  ifdef HAVE_TIME_IN_UTMPX
        utx->ut_time = li->tv_sec;
-#  endif
-#endif
+#  endif /* HAVE_TIME_IN_UTMPX */
+# endif /* HAVE_TV_IN_UTMPX */
 }
 
-
 void
 construct_utmpx(struct logininfo *li, struct utmpx *utx)
 {
-       memset(utx, '\0', sizeof(struct utmpx));
+       memset(utx, '\0', sizeof(*utx));
+# ifdef HAVE_ID_IN_UTMPX
        line_abbrevname(utx->ut_id, li->line, sizeof(utx->ut_id));
+# endif
 
        /* this is done here to keep utmp constants out of loginrec.h */
        switch (li->type) {
@@ -585,57 +675,63 @@ construct_utmpx(struct logininfo *li, struct utmpx *utx)
                utx->ut_type = DEAD_PROCESS;
                break;
        }
-       utx->ut_pid = li->pid;
        line_stripname(utx->ut_line, li->line, sizeof(utx->ut_line));
-       strlcpy(utx->ut_name, li->username, sizeof(utx->ut_name));
        set_utmpx_time(li, utx);
-#ifdef HAVE_HOST_IN_UTMPX
-       strlcpy(utx->ut_host, li->hostname, sizeof(utx->ut_host));
-#endif
-#ifdef HAVE_ADDR_IN_UTMPX
-       /* FIXME: (ATL) not supported yet */
-#endif
-#ifdef HAVE_SYSLEN_IN_UTMPX  
-       /* this is safe because of the extra nulls in logininfo */
-       utx->ut_syslen = strlen(li->hostname);
-#endif
-}
+       utx->ut_pid = li->pid;
 
-#endif
-/* USE_UTMPX || USE_WTMPX */
+       if (li->type == LTYPE_LOGOUT)
+               return;
 
+       /*
+        * These fields are only used when logging in, and are blank
+        * for logouts. 
+        */
 
+       /* strncpy(): Don't necessarily want null termination */
+       strncpy(utx->ut_name, li->username, MIN_SIZEOF(utx->ut_name, li->username));
+# ifdef HAVE_HOST_IN_UTMPX
+       strncpy(utx->ut_host, li->hostname, MIN_SIZEOF(utx->ut_host, li->hostname));
+# endif
+# ifdef HAVE_ADDR_IN_UTMPX
+       /* this is just a 32-bit IP address */
+       if (li->hostaddr.sa.sa_family == AF_INET)
+               utx->ut_addr = li->hostaddr.sa_in.sin_addr.s_addr;
+# endif
+# ifdef HAVE_SYSLEN_IN_UTMPX
+       /* ut_syslen is the length of the utx_host string */
+       utx->ut_syslen = MIN(strlen(li->hostname), sizeof(utx->ut_host));
+# endif
+}
+#endif /* USE_UTMPX || USE_WTMPX */
 
 /**
  ** Low-level utmp functions
  **/
 
 /* FIXME: (ATL) utmp_write_direct needs testing */
-
 #ifdef USE_UTMP
 
 /* if we can, use pututline() etc. */
-#if !defined(DISABLE_PUTUTLINE) && defined(HAVE_SETUTENT) && \
-    defined(HAVE_PUTUTLINE)
+# if !defined(DISABLE_PUTUTLINE) && defined(HAVE_SETUTENT) && \
+       defined(HAVE_PUTUTLINE)
 #  define UTMP_USE_LIBRARY
-#endif
+# endif
 
 
 /* write a utmp entry with the system's help (pututline() and pals) */
-#ifdef UTMP_USE_LIBRARY
+# ifdef UTMP_USE_LIBRARY
 static int
 utmp_write_library(struct logininfo *li, struct utmp *ut)
 {
        setutent();
        pututline(ut);
 
-#ifdef HAVE_ENDUTENT
+#  ifdef HAVE_ENDUTENT
        endutent();
-#endif
+#  endif
        return 1;
 }
-
-#else
+# else /* UTMP_USE_LIBRARY */
 
 /* write a utmp entry direct to the file */
 /* This is a slightly modification of code in OpenBSD's login.c */
@@ -646,8 +742,31 @@ utmp_write_direct(struct logininfo *li, struct utmp *ut)
        register int fd;
        int tty;
 
+       /* FIXME: (ATL) ttyslot() needs local implementation */
+
+#if defined(HAVE_GETTTYENT)
+       register struct ttyent *ty;
+
+       tty=0;
+
+       setttyent();
+       while ((struct ttyent *)0 != (ty = getttyent())) {
+               tty++;
+               if (!strncmp(ty->ty_name, ut->ut_line, sizeof(ut->ut_line)))
+                       break;
+       }
+       endttyent();
+
+       if((struct ttyent *)0 == ty) {
+               log("utmp_write_entry: tty not found");
+               return(1);
+       }
+#else /* FIXME */
+
        tty = ttyslot(); /* seems only to work for /dev/ttyp? style names */
 
+#endif /* HAVE_GETTTYENT */
+
        if (tty > 0 && (fd = open(UTMP_FILE, O_RDWR|O_CREAT, 0644)) >= 0) {
                (void)lseek(fd, (off_t)(tty * sizeof(struct utmp)), SEEK_SET);
                /*
@@ -655,25 +774,25 @@ utmp_write_direct(struct logininfo *li, struct utmp *ut)
                 * If the new ut_line is empty but the old one is not
                 * and ut_line and ut_name match, preserve the old ut_line.
                 */
-               if (   read(fd, &old_ut, sizeof(struct utmp)) == sizeof(struct utmp)
-                      && ut->ut_host[0] == '\0'
-                      && old_ut.ut_host[0] != '\0'
-                      && strncmp(old_ut.ut_line, ut->ut_line, sizeof(ut->ut_line)) == 0
-                      && strncmp(old_ut.ut_name, ut->ut_name, sizeof(ut->ut_name)) == 0  )
+               if (atomicio(read, fd, &old_ut, sizeof(old_ut)) == sizeof(old_ut) && 
+                       (ut->ut_host[0] == '\0') && (old_ut.ut_host[0] != '\0') && 
+                       (strncmp(old_ut.ut_line, ut->ut_line, sizeof(ut->ut_line)) == 0) && 
+                       (strncmp(old_ut.ut_name, ut->ut_name, sizeof(ut->ut_name)) == 0)) {
                        (void)memcpy(ut->ut_host, old_ut.ut_host, sizeof(ut->ut_host));
-
+               }
+               
                (void)lseek(fd, (off_t)(tty * sizeof(struct utmp)), SEEK_SET);
-               if (write(fd, ut, sizeof(struct utmp))==-1)
+               if (atomicio(write, fd, ut, sizeof(*ut)) != sizeof(*ut))
                        log("utmp_write_direct: error writing %s: %s",
-                                    UTMP_FILE, strerror(errno));
+                           UTMP_FILE, strerror(errno));
       
                (void)close(fd);
                return 1;
-       } else
+       } else {
                return 0;
+       }
 }
-#endif /* UTMP_USE_LIBRARY */
-
+# endif /* UTMP_USE_LIBRARY */
 
 static int
 utmp_perform_login(struct logininfo *li)
@@ -681,17 +800,17 @@ utmp_perform_login(struct logininfo *li)
        struct utmp ut;
 
        construct_utmp(li, &ut);
-#ifdef UTMP_USE_LIBRARY
+# ifdef UTMP_USE_LIBRARY
        if (!utmp_write_library(li, &ut)) {
-         log("utmp_perform_login: utmp_write_library() failed");
+               log("utmp_perform_login: utmp_write_library() failed");
                return 0;
        }
-#else
+# else
        if (!utmp_write_direct(li, &ut)) {
                log("utmp_perform_login: utmp_write_direct() failed");
                return 0;
        }
-#endif
+# endif
        return 1;
 }
 
@@ -701,23 +820,18 @@ utmp_perform_logout(struct logininfo *li)
 {
        struct utmp ut;
 
-       memset(&ut, '\0', sizeof(ut));
-       set_utmp_time(li, &ut);
-       line_stripname(ut.ut_line, li->line, sizeof(ut.ut_line));
-#ifdef HAVE_ID_IN_UTMP
-       line_abbrevname(ut.ut_id, li->line, sizeof(ut.ut_id));
-#endif
-#ifdef HAVE_TYPE_IN_UTMP
-       ut.ut_type = DEAD_PROCESS;
-#endif
-
-#if   !defined(DISABLE_PUTUTLINE) \
-    && defined(HAVE_SETUTENT) && defined(HAVE_PUTUTLINE)
-       utmp_write_library(li, &ut);
-#else
-       utmp_write_direct(li, &ut);
-#endif
-
+       construct_utmp(li, &ut);
+# ifdef UTMP_USE_LIBRARY
+       if (!utmp_write_library(li, &ut)) {
+               log("utmp_perform_logout: utmp_write_library() failed");
+               return 0;
+       }
+# else
+       if (!utmp_write_direct(li, &ut)) {
+               log("utmp_perform_logout: utmp_write_direct() failed");
+               return 0;
+       }
+# endif
        return 1;
 }
 
@@ -737,10 +851,7 @@ utmp_write_entry(struct logininfo *li)
                return 0;
        }
 }
-
-
-#endif
-/* USE_UTMP */
+#endif /* USE_UTMP */
 
 
 /**
@@ -751,29 +862,27 @@ utmp_write_entry(struct logininfo *li)
 #ifdef USE_UTMPX
 
 /* if we have the wherewithall, use pututxline etc. */
-#if !defined(DISABLE_PUTUTXLINE) && defined(HAVE_SETUTXENT) \
-    && defined(HAVE_PUTUTXLINE)
+# if !defined(DISABLE_PUTUTXLINE) && defined(HAVE_SETUTXENT) && \
+       defined(HAVE_PUTUTXLINE)
 #  define UTMPX_USE_LIBRARY
-#endif
+# endif
 
 
 /* write a utmpx entry with the system's help (pututxline() and pals) */
-#ifdef UTMPX_USE_LIBRARY
+# ifdef UTMPX_USE_LIBRARY
 static int
 utmpx_write_library(struct logininfo *li, struct utmpx *utx)
 {
        setutxent();
        pututxline(utx);
 
-#ifdef HAVE_ENDUTXENT
+#  ifdef HAVE_ENDUTXENT
        endutxent();
-#endif
+#  endif
        return 1;
 }
 
-#else
-/* UTMPX_USE_LIBRARY */
-
+# else /* UTMPX_USE_LIBRARY */
 
 /* write a utmp entry direct to the file */
 static int
@@ -782,9 +891,7 @@ utmpx_write_direct(struct logininfo *li, struct utmpx *utx)
        log("utmpx_write_direct: not implemented!");
        return 0;
 }
-
-#endif
-/* UTMPX_USE_LIBRARY */
+# endif /* UTMPX_USE_LIBRARY */
 
 static int
 utmpx_perform_login(struct logininfo *li)
@@ -792,17 +899,17 @@ utmpx_perform_login(struct logininfo *li)
        struct utmpx utx;
 
        construct_utmpx(li, &utx);
-#ifdef UTMPX_USE_LIBRARY
+# ifdef UTMPX_USE_LIBRARY
        if (!utmpx_write_library(li, &utx)) {
                log("utmpx_perform_login: utmp_write_library() failed");
                return 0;
        }
-#else
+# else
        if (!utmpx_write_direct(li, &ut)) {
                log("utmpx_perform_login: utmp_write_direct() failed");
                return 0;
        }
-#endif
+# endif
        return 1;
 }
 
@@ -815,22 +922,21 @@ utmpx_perform_logout(struct logininfo *li)
        memset(&utx, '\0', sizeof(utx));
        set_utmpx_time(li, &utx);
        line_stripname(utx.ut_line, li->line, sizeof(utx.ut_line));
-#ifdef HAVE_ID_IN_UTMPX
+# ifdef HAVE_ID_IN_UTMPX
        line_abbrevname(utx.ut_id, li->line, sizeof(utx.ut_id));
-#endif
-#ifdef HAVE_TYPE_IN_UTMPX
+# endif
+# ifdef HAVE_TYPE_IN_UTMPX
        utx.ut_type = DEAD_PROCESS;
-#endif
+# endif
 
-#ifdef UTMPX_USE_LIBRARY
+# ifdef UTMPX_USE_LIBRARY
        utmpx_write_library(li, &utx);
-#else
+# else
        utmpx_write_direct(li, &utx);
-#endif
+# endif
        return 1;
 }
 
-
 int
 utmpx_write_entry(struct logininfo *li)
 {
@@ -844,10 +950,7 @@ utmpx_write_entry(struct logininfo *li)
                return 0;
        }
 }
-
-
-#endif
-/* USE_UTMPX */
+#endif /* USE_UTMPX */
 
 
 /**
@@ -870,8 +973,7 @@ wtmp_write(struct logininfo *li, struct utmp *ut)
                return 0;
        }
        if (fstat(fd, &buf) == 0) 
-               if (write(fd, (char *)ut, sizeof(struct utmp)) != 
-                   sizeof(struct utmp)) {
+               if (atomicio(write, fd, ut, sizeof(*ut)) != sizeof(*ut)) {
                        ftruncate(fd, buf.st_size);
                        log("wtmp_write: problem writing %s: %s",
                            WTMP_FILE, strerror(errno));
@@ -881,9 +983,9 @@ wtmp_write(struct logininfo *li, struct utmp *ut)
        return ret;
 }
 
-
 static int
-wtmp_perform_login(struct logininfo *li){
+wtmp_perform_login(struct logininfo *li)
+{
        struct utmp ut;
 
        construct_utmp(li, &ut);
@@ -897,17 +999,6 @@ wtmp_perform_logout(struct logininfo *li)
        struct utmp ut;
 
        construct_utmp(li, &ut);
-       /* blank out unnecessary fields */
-       memset(&(ut.ut_name), '\0', sizeof(ut.ut_name));
-#ifdef HAVE_ID_IN_UTMP
-       memset(&(ut.ut_id), '\0', sizeof(ut.ut_id));
-#endif
-#ifdef HAVE_HOST_IN_UTMP
-       memset(&(ut.ut_host), '\0', sizeof(ut.ut_host));
-#endif
-#ifdef HAVE_ADDR_IN_UTMP
-       memset(&(ut.ut_addr), '\0', sizeof(ut.ut_addr));
-#endif
        return wtmp_write(li, &ut);
 }
 
@@ -927,12 +1018,47 @@ wtmp_write_entry(struct logininfo *li)
 }
 
 
+/* Notes on fetching login data from wtmp/wtmpx
+ * 
+ * Logouts are usually recorded with (amongst other things) a blank
+ * username on a given tty line.  However, some systems (HP-UX is one)
+ * leave all fields set, but change the ut_type field to DEAD_PROCESS.
+ *
+ * Since we're only looking for logins here, we know that the username
+ * must be set correctly. On systems that leave it in, we check for
+ * ut_type==USER_PROCESS (indicating a login.)
+ *
+ * Portability: Some systems may set something other than USER_PROCESS
+ * to indicate a login process. I don't know of any as I write. Also,
+ * it's possible that some systems may both leave the username in
+ * place and not have ut_type.
+ */
+
+/* return true if this wtmp entry indicates a login */
+static int
+wtmp_islogin(struct logininfo *li, struct utmp *ut)
+{
+       if (strncmp(li->username, ut->ut_name, 
+               MIN_SIZEOF(li->username, ut->ut_name)) == 0) {
+# ifdef HAVE_TYPE_IN_UTMP
+               if (ut->ut_type & USER_PROCESS)
+                       return 1;
+# else
+               return 1;
+# endif
+       }
+       return 0;
+}
+
 int
 wtmp_get_entry(struct logininfo *li)
 {
        struct stat st;
        struct utmp ut;
-       int fd;
+       int fd, found=0;
+
+       /* Clear the time entries in our logininfo */
+       li->tv_sec = li->tv_usec = 0;
 
        if ((fd = open(WTMP_FILE, O_RDONLY)) < 0) {
                log("wtmp_get_entry: problem opening %s: %s",
@@ -945,43 +1071,53 @@ wtmp_get_entry(struct logininfo *li)
                close(fd);
                return 0;
        }
-       (void)lseek(fd, (off_t)(0-sizeof(struct utmp)), SEEK_END);
 
-       do {
-               if (read(fd, &ut, sizeof(ut)) != sizeof(ut)) {
+       /* Seek to the start of the last struct utmp */
+       if (lseek(fd, (off_t)(0 - sizeof(struct utmp)), SEEK_END) == -1) {
+               /* Looks like we've got a fresh wtmp file */
+               close(fd);
+               return 0;
+       }
+
+       while (!found) {
+               if (atomicio(read, fd, &ut, sizeof(ut)) != sizeof(ut)) {
                        log("wtmp_get_entry: read of %s failed: %s",
                            WTMP_FILE, strerror(errno));
                        close (fd);
                        return 0;
                }
-               /* Logouts are recorded as a blank username on a particular line.
-                * So, we just need to find the username in struct utmp */
-               if ( strncmp(li->username, ut.ut_user, 8) == 0 ) {
-                       /* note we've already made sure there's a time in struct utmp */
-#ifdef HAVE_TIME_IN_UTMP
+               if ( wtmp_islogin(li, &ut) ) {
+                       found = 1;
+                       /* We've already checked for a time in struct
+                        * utmp, in login_getlast(). */
+# ifdef HAVE_TIME_IN_UTMP
                        li->tv_sec = ut.ut_time;
-#else
+# else
 #  if HAVE_TV_IN_UTMP
                        li->tv_sec = ut.ut_tv.tv_sec;
 #  endif
-#endif
-                       line_fullname(li->line, ut.ut_line, sizeof(ut.ut_line));
-#ifdef HAVE_HOST_IN_UTMP
-                       strlcpy(li->hostname, ut.ut_host, sizeof(ut.ut_host));
-#endif
+# endif
+                       line_fullname(li->line, ut.ut_line,
+                                     MIN_SIZEOF(li->line, ut.ut_line));
+# ifdef HAVE_HOST_IN_UTMP
+                       strlcpy(li->hostname, ut.ut_host,
+                               MIN_SIZEOF(li->hostname, ut.ut_host));
+# endif
+                       continue;
                }
+               /* Seek back 2 x struct utmp */
                if (lseek(fd, (off_t)(0-2*sizeof(struct utmp)), SEEK_CUR) == -1) {
+                       /* We've found the start of the file, so quit */
                        close (fd);
                        return 0;
                }
-       } while (li->tv_sec == 0);
-      
+       }
+
+       /* We found an entry. Tidy up and return */
+       close(fd);
        return 1;
 }
-
-
-#endif
-/* USE_WTMP */
+# endif /* USE_WTMP */
 
 
 /**
@@ -989,7 +1125,6 @@ wtmp_get_entry(struct logininfo *li)
  **/
 
 #ifdef USE_WTMPX
-
 /* write a wtmpx entry direct to the end of the file */
 /* This is a slight modification of code in OpenBSD's logwtmp.c */
 static int
@@ -1005,8 +1140,7 @@ wtmpx_write(struct logininfo *li, struct utmpx *utx)
        }
 
        if (fstat(fd, &buf) == 0) 
-               if (write(fd, (char *)utx, sizeof(struct utmpx)) != 
-                   sizeof(struct utmpx)) {
+               if (atomicio(write, fd, utx, sizeof(*utx)) != sizeof(*utx)) {
                        ftruncate(fd, buf.st_size);
                        log("wtmpx_write: problem writing %s: %s",
                            WTMPX_FILE, strerror(errno));
@@ -1034,19 +1168,7 @@ wtmpx_perform_logout(struct logininfo *li)
        struct utmpx utx;
 
        construct_utmpx(li, &utx);
-       /* blank out unnecessary fields */
-       memset(&(utx.ut_name), '\0', sizeof(utx.ut_name));
-#ifdef HAVE_ID_IN_UTMPX
-       memset(&(utx.ut_id), '\0', sizeof(utx.ut_id));
-#endif
-#ifdef HAVE_HOST_IN_UTMPX
-       memset(&(utx.ut_host), '\0', sizeof(utx.ut_host));
-#endif
-#ifdef HAVE_ADDR_IN_UTMPX
-       memset(&(utx.ut_addr), '\0', sizeof(utx.ut_addr));
-#endif
        return wtmpx_write(li, &utx);
-
 }
 
 
@@ -1064,13 +1186,35 @@ wtmpx_write_entry(struct logininfo *li)
        }
 }
 
+/* Please see the notes above wtmp_islogin() for information about the
+   next two functions */
+
+/* Return true if this wtmpx entry indicates a login */
+static int
+wtmpx_islogin(struct logininfo *li, struct utmpx *utx)
+{
+       if ( strncmp(li->username, utx->ut_name,
+               MIN_SIZEOF(li->username, utx->ut_name)) == 0 ) {
+# ifdef HAVE_TYPE_IN_UTMPX
+               if (utx->ut_type == USER_PROCESS)
+                       return 1;
+# else
+               return 1;
+# endif
+       }
+       return 0;
+}
+
 
 int
 wtmpx_get_entry(struct logininfo *li)
 {
        struct stat st;
        struct utmpx utx;
-       int fd;
+       int fd, found=0;
+
+       /* Clear the time entries */
+       li->tv_sec = li->tv_usec = 0;
 
        if ((fd = open(WTMPX_FILE, O_RDONLY)) < 0) {
                log("wtmpx_get_entry: problem opening %s: %s",
@@ -1083,10 +1227,16 @@ wtmpx_get_entry(struct logininfo *li)
                close(fd);
                return 0;
        }
-       (void)lseek(fd, (off_t)(0-sizeof(struct utmpx)), SEEK_END);
+       
+       /* Seek to the start of the last struct utmpx */
+       if (lseek(fd, (off_t)(0-sizeof(struct utmpx)), SEEK_END) == -1 ) {
+               /* probably a newly rotated wtmpx file */
+               close(fd);
+               return 0;
+       }
 
-       do {
-               if (read(fd, &utx, sizeof(utx)) != sizeof(utx)) {
+       while (!found) {
+               if (atomicio(read, fd, &utx, sizeof(utx)) != sizeof(utx)) {
                        log("wtmpx_get_entry: read of %s failed: %s",
                            WTMPX_FILE, strerror(errno));
                        close (fd);
@@ -1094,46 +1244,43 @@ wtmpx_get_entry(struct logininfo *li)
                }
                /* Logouts are recorded as a blank username on a particular line.
                 * So, we just need to find the username in struct utmpx */
-               if ( strncmp(li->username, utx.ut_user, 8) == 0 ) {
-                       /* note we've already made sure there's a time in struct utmp */
-#ifdef HAVE_TV_IN_UTMPX
+               if ( wtmpx_islogin(li, &utx) ) {
+# ifdef HAVE_TV_IN_UTMPX
                        li->tv_sec = utx.ut_tv.tv_sec;
-#else
+# else
 #  ifdef HAVE_TIME_IN_UTMPX
                        li->tv_sec = utx.ut_time;
 #  endif
-#endif
-                       line_fullname(li->line, utx.ut_line, sizeof(utx.ut_line));
-#ifdef HAVE_HOST_IN_UTMPX
-                       strlcpy(li->hostname, utx.ut_host, sizeof(utx.ut_line));
-#endif
+# endif
+                       line_fullname(li->line, utx.ut_line, sizeof(li->line));
+# ifdef HAVE_HOST_IN_UTMPX
+                       strlcpy(li->hostname, utx.ut_host,
+                               MIN_SIZEOF(li->hostname, utx.ut_host));
+# endif
+                       continue;
                }
                if (lseek(fd, (off_t)(0-2*sizeof(struct utmpx)), SEEK_CUR) == -1) {
                        close (fd);
                        return 0;
                }
-       } while (li->tv_sec == 0);
+       }
+
+       close(fd);
        return 1;
 }
-
-
-#endif
-/* USE_WTMPX */
-
-
+#endif /* USE_WTMPX */
 
 /**
  ** Low-level libutil login() functions
  **/
 
 #ifdef USE_LOGIN
-
 static int
 syslogin_perform_login(struct logininfo *li)
 {
        struct utmp *ut;
 
-       if (! (ut = (struct utmp *)malloc(sizeof(struct utmp)))) {
+       if (! (ut = (struct utmp *)malloc(sizeof(*ut)))) {
                log("syslogin_perform_login: couldn't malloc()");
                return 0;
        }
@@ -1143,11 +1290,10 @@ syslogin_perform_login(struct logininfo *li)
        return 1;
 }
 
-
 static int
 syslogin_perform_logout(struct logininfo *li)
 {
-#ifdef HAVE_LOGOUT
+# ifdef HAVE_LOGOUT
        char line[8];
   
        (void)line_stripname(line, li->line, sizeof(line));
@@ -1157,16 +1303,16 @@ syslogin_perform_logout(struct logininfo *li)
 #  ifdef HAVE_LOGWTMP
        } else {
                logwtmp(line, "", "");
-       }
 #  endif
-       /* TODO: what to do if we have login, but no logout?
-        * what if logout but no logwtmp? All routines are in libutil
-        * so they should all be there, but... */
-#endif
+       }
+       /* FIXME: (ATL - if the need arises) What to do if we have
+        * login, but no logout?  what if logout but no logwtmp? All
+        * routines are in libutil so they should all be there,
+        * but... */
+# endif
        return 1;
 }
 
-
 int
 syslogin_write_entry(struct logininfo *li)
 {
@@ -1180,45 +1326,39 @@ syslogin_write_entry(struct logininfo *li)
                return 0;
        }
 }
-
-
-#endif
-/* USE_LOGIN */
+#endif /* USE_LOGIN */
 
 /* end of file log-syslogin.c */
 
-
 /**
  ** Low-level lastlog functions
  **/
 
 #ifdef USE_LASTLOG
+#define LL_FILE 1
+#define LL_DIR 2
+#define LL_OTHER 3
 
 static void
 lastlog_construct(struct logininfo *li, struct lastlog *last)
 {
        /* clear the structure */
-       memset(last, '\0', sizeof(struct lastlog));
+       memset(last, '\0', sizeof(*last));
   
-       (void)line_stripname(last->ll_line, li->line,
-                            sizeof(last->ll_line));
-       strlcpy(last->ll_host, li->hostname, sizeof(last->ll_host));
+       (void)line_stripname(last->ll_line, li->line, sizeof(last->ll_line));
+       strlcpy(last->ll_host, li->hostname,
+               MIN_SIZEOF(last->ll_host, li->hostname));
        last->ll_time = li->tv_sec;
 }
 
-
-#define LL_FILE 1
-#define LL_DIR 2
-#define LL_OTHER 3
-
 static int
 lastlog_filetype(char *filename)
 {
        struct stat st;
 
-       if ( stat(LASTLOG_FILE, &st) != 0) {
-               log("lastlog_perform_login: Couldn't stat %s: %s",
-                   LASTLOG_FILE, strerror(errno));
+       if (stat(LASTLOG_FILE, &st) != 0) {
+               log("lastlog_perform_login: Couldn't stat %s: %s", LASTLOG_FILE, 
+                       strerror(errno));
                return 0;
        }
        if (S_ISDIR(st.st_mode))
@@ -1240,33 +1380,37 @@ lastlog_openseek(struct logininfo *li, int *fd, int filemode)
 
        type = lastlog_filetype(LASTLOG_FILE);
        switch (type) {
-       case LL_FILE:
-               strlcpy(lastlog_file, LASTLOG_FILE, sizeof(lastlog_file));
-               break;
-       case LL_DIR:
-               snprintf(lastlog_file, sizeof(lastlog_file), "%s/%s",
-                        LASTLOG_FILE, li->username);
-               break;
-       default:
-               log("lastlog_openseek: %.100s is not a file or directory!",
-                   LASTLOG_FILE);
-               return 0;
-       } /* switch */
+               case LL_FILE:
+                       strlcpy(lastlog_file, LASTLOG_FILE, sizeof(lastlog_file));
+                       break;
+               case LL_DIR:
+                       snprintf(lastlog_file, sizeof(lastlog_file), "%s/%s",
+                                LASTLOG_FILE, li->username);
+                       break;
+               default:
+                       log("lastlog_openseek: %.100s is not a file or directory!",
+                           LASTLOG_FILE);
+                       return 0;
+       }
 
        *fd = open(lastlog_file, filemode);
        if ( *fd < 0) {
-               log("lastlog_openseek: Couldn't open %s: %s",
+               debug("lastlog_openseek: Couldn't open %s: %s",
                    lastlog_file, strerror(errno));
                return 0;
        }
-       /* find this uid's offset in the lastlog file */
-       offset = (off_t) ( (long)li->uid * sizeof(struct lastlog));
-
-       if ( lseek(*fd, offset, SEEK_SET) != offset ) {
-               log("lastlog_openseek: %s->lseek(): %s",
-                   lastlog_file, strerror(errno));
-               return 0;
+       
+       if (type == LL_FILE) {
+               /* find this uid's offset in the lastlog file */
+               offset = (off_t) ( (long)li->uid * sizeof(struct lastlog));
+
+               if ( lseek(*fd, offset, SEEK_SET) != offset ) {
+                       log("lastlog_openseek: %s->lseek(): %s",
+                        lastlog_file, strerror(errno));
+                       return 0;
+               }
        }
+       
        return 1;
 }
 
@@ -1279,19 +1423,20 @@ lastlog_perform_login(struct logininfo *li)
        /* create our struct lastlog */
        lastlog_construct(li, &last);
 
+       if (!lastlog_openseek(li, &fd, O_RDWR|O_CREAT))
+               return(0);
+               
        /* write the entry */
-       if (lastlog_openseek(li, &fd, O_RDWR)) {
-               if ( write(fd, &last, sizeof(struct lastlog)) 
-                    != sizeof(struct lastlog) ) {
-                       log("lastlog_write_filemode: Error writing to %s: %s",
-                           LASTLOG_FILE, strerror(errno));
-                       return 0;
-               }
-               return 1;
-       } else
+       if (atomicio(write, fd, &last, sizeof(last)) != sizeof(last)) {
+               close(fd);
+               log("lastlog_write_filemode: Error writing to %s: %s",
+                   LASTLOG_FILE, strerror(errno));
                return 0;
-}
+       }
 
+       close(fd);
+       return 1;
+}
 
 int
 lastlog_write_entry(struct logininfo *li)
@@ -1305,16 +1450,15 @@ lastlog_write_entry(struct logininfo *li)
        }
 }
 
-
 static void
 lastlog_populate_entry(struct logininfo *li, struct lastlog *last)
 {
        line_fullname(li->line, last->ll_line, sizeof(li->line));
-       strlcpy(li->hostname, last->ll_host, sizeof(li->hostname));
+       strlcpy(li->hostname, last->ll_host, 
+               MIN_SIZEOF(li->hostname, last->ll_host));
        li->tv_sec = last->ll_time;
 }
 
-
 int
 lastlog_get_entry(struct logininfo *li)
 {
@@ -1322,19 +1466,16 @@ lastlog_get_entry(struct logininfo *li)
        int fd;
 
        if (lastlog_openseek(li, &fd, O_RDONLY)) {
-               if ( read(fd, &last, sizeof(struct lastlog)) 
-                    != sizeof(struct lastlog) ) {
-                       log("lastlog_write_filemode: Error reading from %s: %s",
+               if (atomicio(read, fd, &last, sizeof(last)) != sizeof(last)) {
+                       log("lastlog_get_entry: Error reading from %s: %s",
                            LASTLOG_FILE, strerror(errno));
                        return 0;
                } else {
                        lastlog_populate_entry(li, &last);
                        return 1;
                }
-       } else
+       } else {
                return 0;    
+       }
 }
-
-
-#endif
-/* USE_LASTLOG */
+#endif /* USE_LASTLOG */
This page took 0.096791 seconds and 4 git commands to generate.