Dear all,

the following patch, pushed a few minutes ago, identifies
two assignments in "libinetutils/utmp_logout.c" and depending
on a set macro UTMPX. These were written without a variable,
just naming the fields "tv_sec" and "tv_usec" inside "struct timeval".

This being identified now and mended, has as a pleasant consequence
that all our libraries are now building on NexentaCore, i.e.,
an architecture "i386-pc-solaris2.11". A futher patch of mine has
been tested to allow also "syslogd" and "logger" to build successfully
on this OpenSolaris dialect. This matter will go public within
a short time!

Regards,
  Mats


>From de6d5ba5cf6cd3757ff08e62aaeb2e301d634dcd Mon Sep 17 00:00:00 2001
From: Mats Erik Andersson <[email protected]>
Date: Thu, 18 Aug 2011 03:04:26 +0200
Subject: [PATCH] libinetutils.a: Missing variable in UTMPX code.

---
 ChangeLog                  |    5 +++++
 libinetutils/utmp_logout.c |    4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 40cd3ec..b71a316 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2011-08-18  Mats Erik Andersson <[email protected]>
+
+       * libinetutils/utmp_logout.c (utmp_logout) [UTMPX]: Insert missing
+       variable TV in right-hand side.
+
 2011-08-17  Giuseppe Scrivano  <[email protected]>
 
        * bootstrap.conf (gnulib_modules): Use snippet/unused-parameter instead
diff --git a/libinetutils/utmp_logout.c b/libinetutils/utmp_logout.c
index e954c63..fed296f 100644
--- a/libinetutils/utmp_logout.c
+++ b/libinetutils/utmp_logout.c
@@ -76,8 +76,8 @@ utmp_logout (char *line)
       ut->ut_exit.e_termination = 0;
       ut->ut_exit.e_exit = 0;
       gettimeofday (&tv, 0);
-      ut->ut_tv.tv_sec = tv_sec;
-      ut->ut_tv.tv_usec = tv_usec;
+      ut->ut_tv.tv_sec = tv.tv_sec;
+      ut->ut_tv.tv_usec = tv.tv_usec;
       pututxline (ut);
       updwtmpx (PATH_WTMPX, ut);
     }
-- 
1.7.5.4


Reply via email to