]> git.ipfire.org Git - thirdparty/util-linux.git/blobdiff - login-utils/last.c
last: fix wtmp user name buffer overflow [asan]
[thirdparty/util-linux.git] / login-utils / last.c
index dbfa8aed12cc77f4df1ca35b3740b8b98de7d47a..be744b079198ecf11ba05eed990c5f942bca18c0 100644 (file)
  */
 #include <sys/types.h>
 #include <sys/stat.h>
-#include <sys/fcntl.h>
+#include <fcntl.h>
 #include <time.h>
 #include <stdio.h>
 #include <ctype.h>
-#include <utmp.h>
+#include <utmpx.h>
 #include <pwd.h>
 #include <stdlib.h>
 #include <unistd.h>
@@ -40,6 +40,7 @@
 #include <netinet/in.h>
 #include <netdb.h>
 #include <arpa/inet.h>
+#include <libgen.h>
 
 #include "c.h"
 #include "nls.h"
 #include "timeutils.h"
 #include "monotonic.h"
 
-#if defined(_HAVE_UT_TV)
-# define UL_UT_TIME ut_tv.tv_sec
-#else
-# define UL_UT_TIME ut_time
-#endif
-
 #ifndef SHUTDOWN_TIME
 # define SHUTDOWN_TIME 254
 #endif
@@ -99,7 +94,7 @@ struct last_control {
 
 /* Double linked list of struct utmp's */
 struct utmplist {
-       struct utmp ut;
+       struct utmpx ut;
        struct utmplist *next;
        struct utmplist *prev;
 };
@@ -117,22 +112,44 @@ enum {
 
 enum {
        LAST_TIMEFTM_NONE = 0,
-       LAST_TIMEFTM_SHORT_CTIME,
-       LAST_TIMEFTM_FULL_CTIME,
-       LAST_TIMEFTM_ISO8601
+       LAST_TIMEFTM_SHORT,
+       LAST_TIMEFTM_CTIME,
+       LAST_TIMEFTM_ISO8601,
+
+       LAST_TIMEFTM_HHMM,      /* non-public */
 };
 
 struct last_timefmt {
        const char *name;
-       int in;
-       int out;
+       int in_len;     /* log-in */
+       int in_fmt;
+       int out_len;    /* log-out */
+       int out_fmt;
 };
 
 static struct last_timefmt timefmts[] = {
-       [LAST_TIMEFTM_NONE]        = { "notime", 0, 0 },
-       [LAST_TIMEFTM_SHORT_CTIME] = { "short", 16, 7},
-       [LAST_TIMEFTM_FULL_CTIME]  = { "full",  24, 26},
-       [LAST_TIMEFTM_ISO8601]     = { "iso", 24, 26}
+       [LAST_TIMEFTM_NONE] = { .name = "notime" },
+       [LAST_TIMEFTM_SHORT] = {
+               .name    = "short",
+               .in_len  = 16,
+               .out_len = 7,
+               .in_fmt  = LAST_TIMEFTM_CTIME,
+               .out_fmt = LAST_TIMEFTM_HHMM
+       },
+       [LAST_TIMEFTM_CTIME] = {
+               .name    = "full",
+               .in_len  = 24,
+               .out_len = 26,
+               .in_fmt  = LAST_TIMEFTM_CTIME,
+               .out_fmt = LAST_TIMEFTM_CTIME
+       },
+       [LAST_TIMEFTM_ISO8601] = {
+               .name    = "iso",
+               .in_len  = 25,
+               .out_len = 27,
+               .in_fmt  = LAST_TIMEFTM_ISO8601,
+               .out_fmt = LAST_TIMEFTM_ISO8601
+       }
 };
 
 /* Global variables */
@@ -141,22 +158,22 @@ static time_t lastdate;           /* Last date we've seen */
 static time_t currentdate;     /* date when we started processing the file */
 
 /* --time-format=option parser */
-static int which_time_format(const char *optarg)
+static int which_time_format(const char *s)
 {
        size_t i;
 
        for (i = 0; i < ARRAY_SIZE(timefmts); i++) {
-               if (strcmp(timefmts[i].name, optarg) == 0)
+               if (strcmp(timefmts[i].name, s) == 0)
                        return i;
        }
-       errx(EXIT_FAILURE, _("unknown time format: %s"), optarg);
+       errx(EXIT_FAILURE, _("unknown time format: %s"), s);
 }
 
 /*
  *     Read one utmp entry, return in new format.
  *     Automatically reposition file pointer.
  */
-static int uread(FILE *fp, struct utmp *u,  int *quit, const char *filename)
+static int uread(FILE *fp, struct utmpx *u,  int *quit, const char *filename)
 {
        static int utsize;
        static char buf[UCHUNKSIZE];
@@ -169,14 +186,14 @@ static int uread(FILE *fp, struct utmp *u,  int *quit, const char *filename)
                /*
                 *      Normal read.
                 */
-               return fread(u, sizeof(struct utmp), 1, fp);
+               return fread(u, sizeof(struct utmpx), 1, fp);
        }
 
        if (u == NULL) {
                /*
                 *      Initialize and position.
                 */
-               utsize = sizeof(struct utmp);
+               utsize = sizeof(struct utmpx);
                fseeko(fp, 0, SEEK_END);
                fpos = ftello(fp);
                if (fpos == 0)
@@ -200,7 +217,7 @@ static int uread(FILE *fp, struct utmp *u,  int *quit, const char *filename)
         */
        bpos -= utsize;
        if (bpos >= 0) {
-               memcpy(u, buf + bpos, sizeof(struct utmp));
+               memcpy(u, buf + bpos, sizeof(struct utmpx));
                return 1;
        }
 
@@ -236,7 +253,7 @@ static int uread(FILE *fp, struct utmp *u,  int *quit, const char *filename)
        memcpy(tmp, buf + UCHUNKSIZE + bpos, -bpos);
        bpos += UCHUNKSIZE;
 
-       memcpy(u, tmp, sizeof(struct utmp));
+       memcpy(u, tmp, sizeof(struct utmpx));
 
        return 1;
 }
@@ -312,38 +329,27 @@ static int dns_lookup(char *result, int size, int useip, int32_t *a)
        return getnameinfo(sa, salen, result, size, NULL, 0, flags);
 }
 
-static int time_formatter(const struct last_control *ctl, char *dst,
-                         size_t dlen, time_t *when, int pos)
+static int time_formatter(int fmt, char *dst, size_t dlen, time_t *when)
 {
-       struct tm *tm;
        int ret = 0;
 
-       switch (ctl->time_fmt) {
+       switch (fmt) {
        case LAST_TIMEFTM_NONE:
                *dst = 0;
                break;
-       case LAST_TIMEFTM_SHORT_CTIME:
-               if (pos == 0)
-                       ret = sprintf(dst, "%s", ctime(when));
-               else {
-                       tm = localtime(when);
-                       if (!strftime(dst, dlen, "- %H:%M", tm))
-                               ret = -1;
-               }
+       case LAST_TIMEFTM_HHMM:
+       {
+               struct tm *tm = localtime(when);
+               if (!snprintf(dst, dlen, "%02d:%02d", tm->tm_hour, tm->tm_min))
+                       ret = -1;
                break;
-       case LAST_TIMEFTM_FULL_CTIME:
-               if (pos == 0)
-                       ret = sprintf(dst, "%s", ctime(when));
-               else
-                       ret = sprintf(dst, "- %s", ctime(when));
+       }
+       case LAST_TIMEFTM_CTIME:
+               snprintf(dst, dlen, "%s", ctime(when));
+               ret = rtrim_whitespace((unsigned char *) dst);
                break;
        case LAST_TIMEFTM_ISO8601:
-               tm = localtime(when);
-               if (pos == 0) {
-                       if (!strftime(dst, dlen, "%Y-%m-%dT%H:%M:%S%z", tm))
-                               ret = -1;
-               } else if (!strftime(dst, dlen, "- %Y-%m-%dT%H:%M:%S%z", tm))
-                       ret = -1;
+               ret = strtime_iso(when, ISO_TIMESTAMP_T, dst, dlen);
                break;
        default:
                abort();
@@ -371,14 +377,14 @@ static void trim_trailing_spaces(char *s)
 /*
  *     Show one line of information on screen
  */
-static int list(const struct last_control *ctl, struct utmp *p, time_t logout_time, int what)
+static int list(const struct last_control *ctl, struct utmpx *p, time_t logout_time, int what)
 {
        time_t          secs, utmp_time;
        char            logintime[LAST_TIMESTAMP_LEN];
        char            logouttime[LAST_TIMESTAMP_LEN];
        char            length[LAST_TIMESTAMP_LEN];
        char            final[512];
-       char            utline[UT_LINESIZE+1];
+       char            utline[sizeof(p->ut_line) + 1];
        char            domain[256];
        char            *s;
        int             mins, hours, days;
@@ -389,7 +395,7 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
         *      uucp and ftp have special-type entries
         */
        utline[0] = 0;
-       strncat(utline, p->ut_line, UT_LINESIZE);
+       strncat(utline, p->ut_line, sizeof(utline) - 1);
        if (strncmp(utline, "ftp", 3) == 0 && isdigit(utline[3]))
                utline[3] = 0;
        if (strncmp(utline, "uucp", 4) == 0 && isdigit(utline[4]))
@@ -401,7 +407,7 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
        if (ctl->show) {
                char **walk;
                for (walk = ctl->show; *walk; walk++) {
-                       if (strncmp(p->ut_user, *walk, UT_NAMESIZE) == 0 ||
+                       if (strncmp(p->ut_user, *walk, sizeof(p->ut_user)) == 0 ||
                            strcmp(utline, *walk) == 0 ||
                            (strncmp(utline, "tty", 3) == 0 &&
                             strcmp(utline + 3, *walk) == 0)) break;
@@ -412,7 +418,9 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
        /*
         *      Calculate times
         */
-       utmp_time = p->UL_UT_TIME;
+       fmt = &timefmts[ctl->time_fmt];
+
+       utmp_time = p->ut_tv.tv_sec;
 
        if (ctl->present) {
                if (ctl->present < utmp_time)
@@ -420,27 +428,40 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
                if (0 < logout_time && logout_time < ctl->present)
                        return 0;
        }
-       if (time_formatter(ctl, &logintime[0], sizeof(logintime), &utmp_time, 0) < 0 ||
-           time_formatter(ctl, &logouttime[0], sizeof(logouttime), &logout_time, 1) < 0)
+
+       /* log-in time */
+       if (time_formatter(fmt->in_fmt, logintime,
+                          sizeof(logintime), &utmp_time) < 0)
                errx(EXIT_FAILURE, _("preallocation size exceeded"));
 
-       secs  = logout_time - utmp_time;
+       /* log-out time */
+       secs  = logout_time - utmp_time; /* Under strange circumstances, secs < 0 can happen */
        mins  = (secs / 60) % 60;
        hours = (secs / 3600) % 24;
        days  = secs / 86400;
 
+       strcpy(logouttime, "- ");
+       if (time_formatter(fmt->out_fmt, logouttime + 2,
+                          sizeof(logouttime) - 2, &logout_time) < 0)
+               errx(EXIT_FAILURE, _("preallocation size exceeded"));
+
        if (logout_time == currentdate) {
-               if (ctl->time_fmt > LAST_TIMEFTM_SHORT_CTIME) {
+               if (ctl->time_fmt > LAST_TIMEFTM_SHORT) {
                        sprintf(logouttime, "  still running");
                        length[0] = 0;
                } else {
                        sprintf(logouttime, "  still");
                        sprintf(length, "running");
                }
-       } else if (days)
-               sprintf(length, "(%d+%02d:%02d)", days, hours, mins);
-       else
-               sprintf(length, " (%02d:%02d)", hours, mins);
+       } else if (days) {
+               sprintf(length, "(%d+%02d:%02d)", days, abs(hours), abs(mins)); /* hours and mins always shown as positive (w/o minus sign!) even if secs < 0 */
+       } else if (hours) {
+               sprintf(length, " (%02d:%02d)", hours, abs(mins));  /* mins always shown as positive (w/o minus sign!) even if secs < 0 */
+       } else if (secs >= 0) {
+               sprintf(length, " (%02d:%02d)", hours, mins); 
+       } else {
+               sprintf(length, " (-00:%02d)", abs(mins));  /* mins always shown as positive (w/o minus sign!) even if secs < 0 */
+       }
 
        switch(what) {
                case R_CRASH:
@@ -450,7 +471,7 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
                        sprintf(logouttime, "- down ");
                        break;
                case R_NOW:
-                       if (ctl->time_fmt > LAST_TIMEFTM_SHORT_CTIME) {
+                       if (ctl->time_fmt > LAST_TIMEFTM_SHORT) {
                                sprintf(logouttime, "  still logged in");
                                length[0] = 0;
                        } else {
@@ -459,10 +480,10 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
                        }
                        break;
                case R_PHANTOM:
-                       if (ctl->time_fmt > LAST_TIMEFTM_SHORT_CTIME) {
+                       if (ctl->time_fmt > LAST_TIMEFTM_SHORT) {
                                sprintf(logouttime, "  gone - no logout");
                                length[0] = 0;
-                       } else if (ctl->time_fmt == LAST_TIMEFTM_SHORT_CTIME) {
+                       } else if (ctl->time_fmt == LAST_TIMEFTM_SHORT) {
                                sprintf(logouttime, "   gone");
                                sprintf(length, "- no logout");
                        } else {
@@ -470,13 +491,12 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
                                sprintf(length, "no logout");
                        }
                        break;
-               case R_REBOOT:
-                       break;
                case R_TIMECHANGE:
                        logouttime[0] = 0;
                        length[0] = 0;
                        break;
                case R_NORMAL:
+               case R_REBOOT:
                        break;
                default:
                        abort();
@@ -487,15 +507,9 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
         */
        r = -1;
        if (ctl->usedns || ctl->useip)
-               r = dns_lookup(domain, sizeof(domain), ctl->useip, p->ut_addr_v6);
-       if (r < 0) {
-               len = UT_HOSTSIZE;
-               if (len >= (int)sizeof(domain)) len = sizeof(domain) - 1;
-               domain[0] = 0;
-               strncat(domain, p->ut_host, len);
-       }
-
-       fmt = &timefmts[ctl->time_fmt];
+               r = dns_lookup(domain, sizeof(domain), ctl->useip, (int32_t*)p->ut_addr_v6);
+       if (r < 0)
+               mem2strcpy(domain, p->ut_host, sizeof(p->ut_host), sizeof(domain));
 
        if (ctl->showhost) {
                if (!ctl->altlist) {
@@ -503,20 +517,20 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
                                "%-8.*s %-12.12s %-16.*s %-*.*s %-*.*s %s\n",
                                ctl->name_len, p->ut_user, utline,
                                ctl->domain_len, domain,
-                               fmt->in, fmt->in, logintime, fmt->out, fmt->out,
+                               fmt->in_len, fmt->in_len, logintime, fmt->out_len, fmt->out_len,
                                logouttime, length);
                } else {
                        len = snprintf(final, sizeof(final),
                                "%-8.*s %-12.12s %-*.*s %-*.*s %-12.12s %s\n",
                                ctl->name_len, p->ut_user, utline,
-                               fmt->in, fmt->in, logintime, fmt->out, fmt->out,
+                               fmt->in_len, fmt->in_len, logintime, fmt->out_len, fmt->out_len,
                                logouttime, length, domain);
                }
        } else
                len = snprintf(final, sizeof(final),
                        "%-8.*s %-12.12s %-*.*s %-*.*s %s\n",
                        ctl->name_len, p->ut_user, utline,
-                       fmt->in, fmt->in, logintime, fmt->out, fmt->out,
+                       fmt->in_len, fmt->in_len, logintime, fmt->out_len, fmt->out_len,
                        logouttime, length);
 
 #if defined(__GLIBC__)
@@ -543,8 +557,9 @@ static int list(const struct last_control *ctl, struct utmp *p, time_t logout_ti
 }
 
 
-static void __attribute__((__noreturn__)) usage(const struct last_control *ctl, FILE *out)
+static void __attribute__((__noreturn__)) usage(const struct last_control *ctl)
 {
+       FILE *out = stdout;
        fputs(USAGE_HEADER, out);
        fprintf(out, _(
                " %s [options] [<username>...] [<tty>...]\n"), program_invocation_short_name);
@@ -571,22 +586,22 @@ static void __attribute__((__noreturn__)) usage(const struct last_control *ctl,
                "                               notime|short|full|iso\n"), out);
 
        fputs(USAGE_SEPARATOR, out);
-       fputs(USAGE_HELP, out);
-       fputs(USAGE_VERSION, out);
-       fprintf(out, USAGE_MAN_TAIL("last(1)"));
+       printf(USAGE_HELP_OPTIONS(22));
+       printf(USAGE_MAN_TAIL("last(1)"));
 
        exit(out == stderr ? EXIT_FAILURE : EXIT_SUCCESS);
 }
 
-static int is_phantom(const struct last_control *ctl, struct utmp *ut)
+static int is_phantom(const struct last_control *ctl, struct utmpx *ut)
 {
        struct passwd *pw;
-       char path[32];
+       char path[sizeof(ut->ut_line) + 16];
        int ret = 0;
 
-       if (ut->UL_UT_TIME < ctl->boot_time.tv_sec)
+       if (ut->ut_tv.tv_sec < ctl->boot_time.tv_sec)
                return 1;
-       pw = getpwnam(ut->ut_name);
+       ut->ut_user[__UT_NAMESIZE - 1] = '\0';
+       pw = getpwnam(ut->ut_user);
        if (!pw)
                return 1;
        sprintf(path, "/proc/%u/loginuid", ut->ut_pid);
@@ -616,9 +631,9 @@ static int is_phantom(const struct last_control *ctl, struct utmp *ut)
 static void process_wtmp_file(const struct last_control *ctl,
                              const char *filename)
 {
-       FILE *fp;               /* Filepointer of wtmp file */
+       FILE *fp;               /* File pointer of wtmp file */
 
-       struct utmp ut;         /* Current utmp entry */
+       struct utmpx ut;        /* Current utmp entry */
        struct utmplist *ulist = NULL;  /* All entries */
        struct utmplist *p;     /* Pointer into utmplist */
        struct utmplist *next;  /* Pointer into utmplist */
@@ -661,7 +676,7 @@ static void process_wtmp_file(const struct last_control *ctl,
         * Read first structure to capture the time field
         */
        if (uread(fp, &ut, NULL, filename) == 1)
-               begintime = ut.UL_UT_TIME;
+               begintime = ut.ut_tv.tv_sec;
        else {
                if (fstat(fileno(fp), &st) != 0)
                        err(EXIT_FAILURE, _("stat of %s failed"), filename);
@@ -683,16 +698,16 @@ static void process_wtmp_file(const struct last_control *ctl,
                if (uread(fp, &ut, &quit, filename) != 1)
                        break;
 
-               if (ctl->since && ut.UL_UT_TIME < ctl->since)
+               if (ctl->since && ut.ut_tv.tv_sec < ctl->since)
                        continue;
 
-               if (ctl->until && ctl->until < ut.UL_UT_TIME)
+               if (ctl->until && ctl->until < ut.ut_tv.tv_sec)
                        continue;
 
-               lastdate = ut.UL_UT_TIME;
+               lastdate = ut.ut_tv.tv_sec;
 
                if (ctl->lastb) {
-                       quit = list(ctl, &ut, ut.UL_UT_TIME, R_NORMAL);
+                       quit = list(ctl, &ut, ut.ut_tv.tv_sec, R_NORMAL);
                        continue;
                }
 
@@ -742,7 +757,7 @@ static void process_wtmp_file(const struct last_control *ctl,
                                strcpy(ut.ut_line, "system down");
                                quit = list(ctl, &ut, lastboot, R_NORMAL);
                        }
-                       lastdown = lastrch = ut.UL_UT_TIME;
+                       lastdown = lastrch = ut.ut_tv.tv_sec;
                        down = 1;
                        break;
                case OLD_TIME:
@@ -757,7 +772,7 @@ static void process_wtmp_file(const struct last_control *ctl,
                case BOOT_TIME:
                        strcpy(ut.ut_line, "system boot");
                        quit = list(ctl, &ut, lastdown, R_REBOOT);
-                       lastboot = ut.UL_UT_TIME;
+                       lastboot = ut.ut_tv.tv_sec;
                        down = 1;
                        break;
                case RUN_LVL:
@@ -767,11 +782,11 @@ static void process_wtmp_file(const struct last_control *ctl,
                                quit = list(ctl, &ut, lastrch, R_NORMAL);
                        }
                        if (x == '0' || x == '6') {
-                               lastdown = ut.UL_UT_TIME;
+                               lastdown = ut.ut_tv.tv_sec;
                                down = 1;
                                ut.ut_type = SHUTDOWN_TIME;
                        }
-                       lastrch = ut.UL_UT_TIME;
+                       lastrch = ut.ut_tv.tv_sec;
                        break;
 
                case USER_PROCESS:
@@ -784,10 +799,10 @@ static void process_wtmp_file(const struct last_control *ctl,
                        for (p = ulist; p; p = next) {
                                next = p->next;
                                if (strncmp(p->ut.ut_line, ut.ut_line,
-                                   UT_LINESIZE) == 0) {
+                                   sizeof(ut.ut_line)) == 0) {
                                        /* Show it */
                                        if (c == 0) {
-                                               quit = list(ctl, &ut, p->ut.UL_UT_TIME, R_NORMAL);
+                                               quit = list(ctl, &ut, p->ut.ut_tv.tv_sec, R_NORMAL);
                                                c = 1;
                                        }
                                        if (p->next)
@@ -813,7 +828,7 @@ static void process_wtmp_file(const struct last_control *ctl,
                                        c = whydown;
                                quit = list(ctl, &ut, lastboot, c);
                        }
-                       /* FALLTHRU */
+                       /* fallthrough */
 
                case DEAD_PROCESS:
                        /*
@@ -823,7 +838,7 @@ static void process_wtmp_file(const struct last_control *ctl,
                        if (ut.ut_line[0] == 0)
                                break;
                        p = xmalloc(sizeof(struct utmplist));
-                       memcpy(&p->ut, &ut, sizeof(struct utmp));
+                       memcpy(&p->ut, &ut, sizeof(struct utmpx));
                        p->next  = ulist;
                        p->prev  = NULL;
                        if (ulist)
@@ -834,12 +849,14 @@ static void process_wtmp_file(const struct last_control *ctl,
                case EMPTY:
                case INIT_PROCESS:
                case LOGIN_PROCESS:
+#ifdef ACCOUNTING
                case ACCOUNTING:
+#endif
                        /* ignored ut_types */
                        break;
 
                default:
-                       warnx("unrecogized ut_type: %d", ut.ut_type);
+                       warnx("unrecognized ut_type: %d", ut.ut_type);
                }
 
                /*
@@ -847,7 +864,7 @@ static void process_wtmp_file(const struct last_control *ctl,
                 * the entire current ulist.
                 */
                if (down) {
-                       lastboot = ut.UL_UT_TIME;
+                       lastboot = ut.ut_tv.tv_sec;
                        whydown = (ut.ut_type == SHUTDOWN_TIME) ? R_DOWN : R_CRASH;
                        for (p = ulist; p; p = next) {
                                next = p->next;
@@ -858,7 +875,19 @@ static void process_wtmp_file(const struct last_control *ctl,
                }
        }
 
-       printf(_("\n%s begins %s"), basename(filename), ctime(&begintime));
+       if (ctl->time_fmt != LAST_TIMEFTM_NONE) {
+               struct last_timefmt *fmt;
+               char timestr[LAST_TIMESTAMP_LEN];
+               char *tmp = xstrdup(filename);
+
+               fmt = &timefmts[ctl->time_fmt];
+               if (time_formatter(fmt->in_fmt, timestr,
+                                  sizeof(timestr), &begintime) < 0)
+                       errx(EXIT_FAILURE, _("preallocation size exceeded"));
+               printf(_("\n%s begins %s\n"), basename(tmp), timestr);
+               free(tmp);
+       }
+
        fclose(fp);
 
        for (p = ulist; p; p = next) {
@@ -872,7 +901,7 @@ int main(int argc, char **argv)
        struct last_control ctl = {
                .showhost = TRUE,
                .name_len = LAST_LOGIN_LEN,
-               .time_fmt = LAST_TIMEFTM_SHORT_CTIME,
+               .time_fmt = LAST_TIMEFTM_SHORT,
                .domain_len = LAST_DOMAIN_LEN
        };
        char **files = NULL;
@@ -901,7 +930,7 @@ int main(int argc, char **argv)
              { "time-format", required_argument, NULL, OPT_TIME_FORMAT },
              { NULL, 0, NULL, 0 }
        };
-       static const ul_excl_t excl[] = {       /* rows and cols in in ASCII order */
+       static const ul_excl_t excl[] = {       /* rows and cols in ASCII order */
                { 'F', OPT_TIME_FORMAT },       /* fulltime, time-format */
                { 0 }
        };
@@ -922,7 +951,7 @@ int main(int argc, char **argv)
 
                switch(c) {
                case 'h':
-                       usage(&ctl, stdout);
+                       usage(&ctl);
                        break;
                case 'V':
                        printf(UTIL_LINUX_VERSION);
@@ -951,7 +980,7 @@ int main(int argc, char **argv)
                        ctl.altlist = 1;
                        break;
                case 'F':
-                       ctl.time_fmt = LAST_TIMEFTM_FULL_CTIME;
+                       ctl.time_fmt = LAST_TIMEFTM_CTIME;
                        break;
                case 'p':
                        if (parse_timestamp(optarg, &p) < 0)
@@ -969,10 +998,10 @@ int main(int argc, char **argv)
                        ctl.until = (time_t) (p / 1000000);
                        break;
                case 'w':
-                       if (ctl.name_len < UT_NAMESIZE)
-                               ctl.name_len = UT_NAMESIZE;
-                       if (ctl.domain_len < UT_HOSTSIZE)
-                               ctl.domain_len = UT_HOSTSIZE;
+                       if (ctl.name_len < sizeof(((struct utmpx *) 0)->ut_user))
+                               ctl.name_len = sizeof(((struct utmpx *) 0)->ut_user);
+                       if (ctl.domain_len < sizeof(((struct utmpx *) 0)->ut_host))
+                               ctl.domain_len = sizeof(((struct utmpx *) 0)->ut_host);
                        break;
                case '0': case '1': case '2': case '3': case '4':
                case '5': case '6': case '7': case '8': case '9':
@@ -982,8 +1011,7 @@ int main(int argc, char **argv)
                        ctl.time_fmt = which_time_format(optarg);
                        break;
                default:
-                       usage(&ctl, stderr);
-                       break;
+                       errtryhelp(EXIT_FAILURE);
                }
        }