hust testsuite: fix a false positive

[oweals/busybox.git] / miscutils / last_fancy.c

diff --git a/miscutils/last_fancy.c b/miscutils/last_fancy.c
index f687d7e16c7c201c4f784624810c2b95c814e67b..e56e0ba85995772ecf5f28e3ea33598e0ee4d223 100644 (file)
--- a/miscutils/last_fancy.c
+++ b/miscutils/last_fancy.c
@@ -22,6 +22,10 @@
 #define HEADER_LINE_WIDE  "USER", "TTY", \
        INET6_ADDRSTRLEN, INET6_ADDRSTRLEN, "HOST", "LOGIN", "  TIME", ""
 
+#if !defined __UT_LINESIZE && defined UT_LINESIZE
+# define __UT_LINESIZE UT_LINESIZE
+#endif
+
 enum {
        NORMAL,
        LOGGED,
@@ -39,10 +43,10 @@ enum {
 
 #define show_wide (option_mask32 & LAST_OPT_W)
 
-static void show_entry(struct utmp *ut, int state, time_t dur_secs)
+static void show_entry(struct utmpx *ut, int state, time_t dur_secs)
 {
        unsigned days, hours, mins;
-       char duration[32];
+       char duration[sizeof("(%u+02:02)") + sizeof(int)*3];
        char login_time[17];
        char logout_time[8];
        const char *logout_str;
@@ -53,7 +57,8 @@ static void show_entry(struct utmp *ut, int state, time_t dur_secs)
         * but some systems have it wrong */
        tmp = ut->ut_tv.tv_sec;
        safe_strncpy(login_time, ctime(&tmp), 17);
-       snprintf(logout_time, 8, "- %s", ctime(&dur_secs) + 11);
+       tmp = dur_secs;
+       snprintf(logout_time, 8, "- %s", ctime(&tmp) + 11);
 
        dur_secs = MAX(dur_secs - (time_t)ut->ut_tv.tv_sec, (time_t)0);
        /* unsigned int is easier to divide than time_t (which may be signed long) */
@@ -103,7 +108,7 @@ static void show_entry(struct utmp *ut, int state, time_t dur_secs)
                duration_str);
 }
 
-static int get_ut_type(struct utmp *ut)
+static int get_ut_type(struct utmpx *ut)
 {
        if (ut->ut_line[0] == '~') {
                if (strcmp(ut->ut_user, "shutdown") == 0) {
@@ -141,7 +146,7 @@ static int get_ut_type(struct utmp *ut)
        return ut->ut_type;
 }
 
-static int is_runlevel_shutdown(struct utmp *ut)
+static int is_runlevel_shutdown(struct utmpx *ut)
 {
        if (((ut->ut_pid & 255) == '0') || ((ut->ut_pid & 255) == '6')) {
                return 1;
@@ -153,7 +158,7 @@ static int is_runlevel_shutdown(struct utmp *ut)
 int last_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int last_main(int argc UNUSED_PARAM, char **argv)
 {
-       struct utmp ut;
+       struct utmpx ut;
        const char *filename = _PATH_WTMP;
        llist_t *zlist;
        off_t pos;
@@ -228,7 +233,7 @@ int last_main(int argc UNUSED_PARAM, char **argv)
                                break;
                        }
                        /* add_entry */
-                       llist_add_to(&zlist, memcpy(xmalloc(sizeof(ut)), &ut, sizeof(ut)));
+                       llist_add_to(&zlist, xmemdup(&ut, sizeof(ut)));
                        break;
                case USER_PROCESS: {
                        int show;
@@ -241,9 +246,9 @@ int last_main(int argc UNUSED_PARAM, char **argv)
                        {
                                llist_t *el, *next;
                                for (el = zlist; el; el = next) {
-                                       struct utmp *up = (struct utmp *)el->data;
+                                       struct utmpx *up = (struct utmpx *)el->data;
                                        next = el->link;
-                                       if (strncmp(up->ut_line, ut.ut_line, UT_LINESIZE) == 0) {
+                                       if (strncmp(up->ut_line, ut.ut_line, __UT_LINESIZE) == 0) {
                                                if (show) {
                                                        show_entry(&ut, NORMAL, up->ut_tv.tv_sec);
                                                        show = 0;
@@ -270,7 +275,7 @@ int last_main(int argc UNUSED_PARAM, char **argv)
                                show_entry(&ut, state, boot_time);
                        }
                        /* add_entry */
-                       llist_add_to(&zlist, memcpy(xmalloc(sizeof(ut)), &ut, sizeof(ut)));
+                       llist_add_to(&zlist, xmemdup(&ut, sizeof(ut)));
                        break;
                }
                }