diff options
| author | Gerrit Pape <pape@smarden.org> | 2014-07-31 12:25:49 +0000 |
|---|---|---|
| committer | Gerrit Pape <pape@smarden.org> | 2024-09-25 23:38:34 +0000 |
| commit | 790997147bf6b6c34023f8328a87cbeb7a97324d (patch) | |
| tree | e57950cbe9a9d8b2855c0ad28e8b7d1cc730433a /src/utmpset.c | |
| parent | 47aa117b0ee6ffcf9aa398bd343e7e3491791238 (diff) | |
| download | runit-790997147bf6b6c34023f8328a87cbeb7a97324d.tar.gz runit-790997147bf6b6c34023f8328a87cbeb7a97324d.tar.xz runit-790997147bf6b6c34023f8328a87cbeb7a97324d.zip | |
utmpset.c: mixes "int32_t *" and "time_t *"
Don't pass int32_t to time(), thx Lorenzo Beretta, https://bugs.debian.org/754849
Diffstat (limited to 'src/utmpset.c')
| -rw-r--r-- | src/utmpset.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/utmpset.c b/src/utmpset.c index 05ccb10..6ced914 100644 --- a/src/utmpset.c +++ b/src/utmpset.c @@ -24,6 +24,7 @@ void usage(void) { strerr_die4x(1, "usage: ", progname, USAGE, "\n"); } int utmp_logout(const char *line) { int fd; uw_tmp ut; + time_t t; int ok =-1; if ((fd =open(UW_TMP_UFILE, O_RDWR, 0)) < 0) @@ -35,7 +36,8 @@ int utmp_logout(const char *line) { if (!ut.ut_name[0] || (str_diff(ut.ut_line, line) != 0)) continue; memset(ut.ut_name, 0, sizeof ut.ut_name); memset(ut.ut_host, 0, sizeof ut.ut_host); - if (time(&ut.ut_time) == -1) break; + if (time(&t) == -1) break; + ut.ut_time = t; #ifdef DEAD_PROCESS ut.ut_type =DEAD_PROCESS; #endif @@ -52,6 +54,7 @@ int wtmp_logout(const char *line) { int len; struct stat st; uw_tmp ut; + time_t t; if ((fd = open_append(UW_TMP_WFILE)) == -1) strerr_die4sys(111, FATAL, "unable to open ", UW_TMP_WFILE, ": "); @@ -65,10 +68,11 @@ int wtmp_logout(const char *line) { memset(&ut, 0, sizeof(uw_tmp)); if ((len =str_len(line)) > sizeof ut.ut_line) len =sizeof ut.ut_line -2; byte_copy(ut.ut_line, len, line); - if (time(&ut.ut_time) == -1) { + if (time(&t) == -1) { close(fd); return(-1); } + ut.ut_time = t; #ifdef DEAD_PROCESS ut.ut_type =DEAD_PROCESS; #endif |