--- rxvt-unicode/src/logging.C	2004/04/08 20:31:45	1.9
+++ rxvt-unicode/src/logging.C	2005/11/08 17:35:28	1.18
@@ -44,6 +44,13 @@
 #include "logging.h"
 #ifdef UTMP_SUPPORT
 
+#if HAVE_STRUCT_UTMP
+int              rxvt_write_bsd_utmp              (int utmp_pos, struct utmp *wu);
+void             rxvt_update_wtmp                 (const char *fname, const struct utmp *putmp);
+#endif
+
+void             rxvt_update_lastlog              (const char *fname, const char *pty, const char *host);
+
 /*
  * BSD style utmp entry
  *      ut_line, ut_name, ut_host, ut_time
@@ -60,95 +67,94 @@
 rxvt_term::makeutent (const char *pty, const char *hostname)
 {
 #ifdef HAVE_STRUCT_UTMP
-  struct utmp    *ut = & (this->ut);
+  struct utmp *ut = &this->ut;
 #endif
 #ifdef HAVE_STRUCT_UTMPX
-  struct utmpx   *utx = & (this->utx);
+  struct utmpx *utx = &this->utx;
 #endif
 #ifdef HAVE_UTMP_PID
-  int             i;
+  int i;
 #endif
-  char            ut_id[5];
-  struct passwd  *pwent = getpwuid (getuid ());
+  struct passwd *pwent = getpwuid (getuid ());
 
-  if (!STRNCMP (pty, "/dev/", 5))
+  if (!strncmp (pty, "/dev/", 5))
     pty += 5;		/* skip /dev/ prefix */
 
-  if (!STRNCMP (pty, "pty", 3) || !STRNCMP (pty, "tty", 3))
-    {
-      STRNCPY (ut_id, (pty + 3), sizeof (ut_id));
-    }
 #ifdef HAVE_UTMP_PID
+  if (!strncmp (pty, "pty", 3) || !strncmp (pty, "tty", 3))
+    strncpy (ut_id, pty + 3, sizeof (ut_id));
   else if (sscanf (pty, "pts/%d", &i) == 1)
     sprintf (ut_id, "vt%02x", (i & 0xff));	/* sysv naming */
-#endif
-  else if (STRNCMP (pty, "pty", 3) && STRNCMP (pty, "tty", 3))
+  else if (strncmp (pty, "pty", 3) && strncmp (pty, "tty", 3))
     {
       rxvt_warn ("can't parse tty name \"%s\", not adding utmp entry.\n", pty);
       return;
     }
+#endif
 
 #ifdef HAVE_STRUCT_UTMP
-  MEMSET (ut, 0, sizeof (struct utmp));
+  memset (ut, 0, sizeof (struct utmp));
 # ifdef HAVE_UTMP_PID
   setutent ();
-  STRNCPY (ut->ut_id, ut_id, sizeof (ut->ut_id));
+  strncpy (ut->ut_id, ut_id, sizeof (ut->ut_id));
   ut->ut_type = DEAD_PROCESS;
   getutid (ut);		/* position to entry in utmp file */
-  STRNCPY (ut_id, ut_id, sizeof (ut_id));
+  strncpy (ut_id, ut_id, sizeof (ut_id));
 # endif
 #endif
 
 #ifdef HAVE_STRUCT_UTMPX
-  MEMSET (utx, 0, sizeof (struct utmpx));
+  memset (utx, 0, sizeof (struct utmpx));
   setutxent ();
-  STRNCPY (utx->ut_id, ut_id, sizeof (utx->ut_id));
+  strncpy (utx->ut_id, ut_id, sizeof (utx->ut_id));
   utx->ut_type = DEAD_PROCESS;
   getutxid (utx);		/* position to entry in utmp file */
-  STRNCPY (ut_id, ut_id, sizeof (ut_id));
+  strncpy (ut_id, ut_id, sizeof (ut_id));
 #endif
 
 #ifdef HAVE_STRUCT_UTMP
-  STRNCPY (ut->ut_line, pty, sizeof (ut->ut_line));
+  strncpy (ut->ut_line, pty, sizeof (ut->ut_line));
   ut->ut_time = time (NULL);
 # ifdef HAVE_UTMP_PID
-  STRNCPY (ut->ut_user, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
+  strncpy (ut->ut_user, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
           sizeof (ut->ut_user));
-  STRNCPY (ut->ut_id, ut_id, sizeof (ut->ut_id));
+  strncpy (ut->ut_id, ut_id, sizeof (ut->ut_id));
   ut->ut_time = time (NULL);
   ut->ut_pid = cmd_pid;
 #  ifdef HAVE_UTMP_HOST
-  STRNCPY (ut->ut_host, hostname, sizeof (ut->ut_host));
+  strncpy (ut->ut_host, hostname, sizeof (ut->ut_host));
 #  endif
   ut->ut_type = USER_PROCESS;
   pututline (ut);
   endutent ();			/* close the file */
   utmp_pos = 0;
 # else
-  STRNCPY (ut->ut_name, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
+  strncpy (ut->ut_name, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
           sizeof (ut->ut_name));
 #  ifdef HAVE_UTMP_HOST
-  STRNCPY (ut->ut_host, hostname, sizeof (ut->ut_host));
+  strncpy (ut->ut_host, hostname, sizeof (ut->ut_host));
 #  endif
 # endif
 #endif
 
 #ifdef HAVE_STRUCT_UTMPX
-  STRNCPY (utx->ut_line, pty, sizeof (utx->ut_line));
-  STRNCPY (utx->ut_user, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
+  strncpy (utx->ut_line, pty, sizeof (utx->ut_line));
+  strncpy (utx->ut_user, (pwent && pwent->pw_name) ? pwent->pw_name : "?",
           sizeof (utx->ut_user));
-  STRNCPY (utx->ut_id, ut_id, sizeof (utx->ut_id));
+  strncpy (utx->ut_id, ut_id, sizeof (utx->ut_id));
+# if HAVE_UTMPX_SESSION
   utx->ut_session = getsid (0);
+# endif
   utx->ut_tv.tv_sec = time (NULL);
   utx->ut_tv.tv_usec = 0;
   utx->ut_pid = cmd_pid;
 # ifdef HAVE_UTMPX_HOST
-  STRNCPY (utx->ut_host, hostname, sizeof (utx->ut_host));
+  strncpy (utx->ut_host, hostname, sizeof (utx->ut_host));
 #  if 0
   {
     char           *colon;
 
-    if ((colon = STRRCHR (ut->ut_host, ':')) != NULL)
+    if ((colon = strrchr (ut->ut_host, ':')) != NULL)
       *colon = '\0';
   }
 #  endif
@@ -172,14 +178,14 @@
 
     if ((fd0 = fopen (TTYTAB_FILENAME, "r")) != NULL)
       {
-        char            buf[256], name[256];
+        char buf[256], name[256];
 
         buf[sizeof (buf) - 1] = '\0';
         for (i = 1; (fgets (buf, sizeof (buf) - 1, fd0) != NULL);)
           {
             if (*buf == '#' || sscanf (buf, "%s", name) != 1)
               continue;
-            if (!STRCMP (ut->ut_line, name))
+            if (!strcmp (ut->ut_line, name))
               {
                 if (!rxvt_write_bsd_utmp (i, ut))
                   i = 0;
@@ -198,7 +204,7 @@
 
 #ifdef WTMP_SUPPORT
 # ifdef WTMP_ONLY_ON_LOGIN
-  if (Options & Opt_loginShell)
+  if (options & Opt_loginShell)
 # endif
     {
 # ifdef HAVE_STRUCT_UTMP
@@ -209,12 +215,16 @@
 #  endif
 # endif
 # ifdef HAVE_STRUCT_UTMPX
+#  if HAVE_UPDWTMPX
       updwtmpx (RXVT_WTMPX_FILE, utx);
+#  else
+      pututxline (utx);
+#  endif
 # endif
     }
 #endif
 #if defined(LASTLOG_SUPPORT) && defined(RXVT_LASTLOG_FILE)
-  if (Options & Opt_loginShell)
+  if (options & Opt_loginShell)
     rxvt_update_lastlog (RXVT_LASTLOG_FILE, pty, hostname);
 #endif
 }
@@ -227,17 +237,17 @@
 rxvt_term::cleanutent ()
 {
 #ifdef HAVE_STRUCT_UTMP
-  struct utmp    *ut = & (this->ut);
+  struct utmp *ut = &this->ut;
 #endif
 #ifdef HAVE_STRUCT_UTMPX
-  struct utmpx   *tmputx, *utx = & (this->utx);
+  struct utmpx *tmputx, *utx = &this->utx;
 #endif
 
 #ifdef HAVE_STRUCT_UTMP
 # ifdef HAVE_UTMP_PID
-  MEMSET (ut, 0, sizeof (struct utmp));
+  memset (ut, 0, sizeof (struct utmp));
   setutent ();
-  STRNCPY (ut->ut_id, ut_id, sizeof (ut->ut_id));
+  strncpy (ut->ut_id, ut_id, sizeof (ut->ut_id));
   ut->ut_type = USER_PROCESS;
   {
     struct utmp    *tmput = getutid (ut);
@@ -247,23 +257,25 @@
   }
   ut->ut_type = DEAD_PROCESS;
 # else
-  MEMSET (ut->ut_name, 0, sizeof (ut->ut_name));
+  memset (ut->ut_name, 0, sizeof (ut->ut_name));
 #  ifdef HAVE_UTMP_HOST
-  MEMSET (ut->ut_host, 0, sizeof (ut->ut_host));
+  memset (ut->ut_host, 0, sizeof (ut->ut_host));
 #  endif
 # endif
   ut->ut_time = time (NULL);
 #endif
 
 #ifdef HAVE_STRUCT_UTMPX
-  MEMSET (utx, 0, sizeof (struct utmpx));
+  memset (utx, 0, sizeof (struct utmpx));
   setutxent ();
-  STRNCPY (utx->ut_id, ut_id, sizeof (utx->ut_id));
+  strncpy (utx->ut_id, ut_id, sizeof (utx->ut_id));
   utx->ut_type = USER_PROCESS;
   if ((tmputx = getutxid (utx)))	/* position to entry in utmp file */
     utx = tmputx;
   utx->ut_type = DEAD_PROCESS;
+# if HAVE_UTMPX_SESSION
   utx->ut_session = getsid (0);
+# endif
   utx->ut_tv.tv_sec = time (NULL);
   utx->ut_tv.tv_usec = 0;
 #endif
@@ -273,7 +285,7 @@
    */
 #ifdef WTMP_SUPPORT
 # ifdef WTMP_ONLY_ON_LOGIN
-  if (Options & Opt_loginShell)
+  if (options & Opt_loginShell)
 # endif
     {
 # ifdef HAVE_STRUCT_UTMP
@@ -284,7 +296,11 @@
 #  endif
 # endif
 # ifdef HAVE_STRUCT_UTMPX
+#  if HAVE_UPDWTMPX
       updwtmpx (RXVT_WTMPX_FILE, utx);
+#  else
+      pututxline (utx);
+#  endif
 # endif
     }
 #endif
@@ -298,7 +314,7 @@
     pututline (ut);
   endutent ();
 # else
-  MEMSET (ut, 0, sizeof (struct utmp));
+  memset (ut, 0, sizeof (struct utmp));
   rxvt_write_bsd_utmp (utmp_pos, ut);
 # endif
 #endif
@@ -314,7 +330,6 @@
  * Write a BSD style utmp entry
  */
 #ifdef HAVE_UTMP_H
-/* INTPROTO */
 int
 rxvt_write_bsd_utmp (int utmp_pos, struct utmp *wu)
 {
@@ -334,8 +349,7 @@
 /*
  * Update a BSD style wtmp entry
  */
-#if defined(WTMP_SUPPORT) && !defined(HAVE_UPDWTMP)
-/* INTPROTO */
+#if defined(WTMP_SUPPORT) && !defined(HAVE_UPDWTMP) && defined(HAVE_STRUCT_UTMP)
 void
 rxvt_update_wtmp (const char *fname, const struct utmp *putmp)
 {
@@ -377,7 +391,6 @@
 
 /* ------------------------------------------------------------------------- */
 #ifdef LASTLOG_SUPPORT
-/* INTPROTO */
 void
 rxvt_update_lastlog (const char *fname, const char *pty, const char *host)
 {
@@ -394,11 +407,11 @@
 # endif
 
 # ifdef HAVE_STRUCT_LASTLOGX
-  MEMSET (&llx, 0, sizeof (llx));
+  memset (&llx, 0, sizeof (llx));
   llx.ll_tv.tv_sec = time (NULL);
   llx.ll_tv.tv_usec = 0;
-  STRNCPY (llx.ll_line, pty, sizeof (llx.ll_line));
-  STRNCPY (llx.ll_host, host, sizeof (llx.ll_host));
+  strncpy (llx.ll_line, pty, sizeof (llx.ll_line));
+  strncpy (llx.ll_host, host, sizeof (llx.ll_host));
   updlastlogx (RXVT_LASTLOGX_FILE, getuid (), &llx);
 # endif
 
@@ -410,10 +423,10 @@
       return;
     }
 
-  MEMSET (&ll, 0, sizeof (ll));
+  memset (&ll, 0, sizeof (ll));
   ll.ll_time = time (NULL);
-  STRNCPY (ll.ll_line, pty, sizeof (ll.ll_line));
-  STRNCPY (ll.ll_host, host, sizeof (ll.ll_host));
+  strncpy (ll.ll_line, pty, sizeof (ll.ll_line));
+  strncpy (ll.ll_host, host, sizeof (ll.ll_host));
 #  ifdef LASTLOG_IS_DIR
   sprintf (lastlogfile, "%.*s/%.*s",
           sizeof (lastlogfile) - sizeof (pwent->pw_name) - 2, fname,