Changeset 154 for trunk/src/sh_utmp.c

Timestamp:

Jan 12, 2008, 7:04:06 PM (17 years ago)

Author:

katerina

Message:

Fix more compiler warnings, and a potential NULL dereference in the unix entropy collector.

File:

• trunk/src/sh_utmp.c (modified) (9 diffs)

trunk/src/sh_utmp.c

@@ -771 +771 @@
 #endif
 
+/* These variables are not used anywhere. They only exist
+ * to assign &userold, &user to them, which keeps gcc from
+ * putting them into a register, and avoids the 'clobbered
+ * by longjmp' warning. And no, 'volatile' proved insufficient.
+ */
+void * sh_dummy_userold = NULL;
+void * sh_dummy_user    = NULL;
+
+
 static void sh_utmp_addlogin (struct SH_UTMP_S * ut)
 {
-  struct          log_user   * user     = userlist;
-  volatile struct log_user   * userold  = userlist;
+  struct log_user   * user     = userlist;
+  struct log_user   * userold  = userlist;
 #ifdef HAVE_UTTYPE  
   struct log_user   * username = userlist;
@@ -785 +794 @@
 
   SL_ENTER(_("sh_utmp_addlogin"));
+
+  /* Take the address to keep gcc from putting them into registers. 
+   * Avoids the 'clobbered by longjmp' warning. 
+   */
+  sh_dummy_userold = (void*) &userold;
+  sh_dummy_user    = (void*) &user;
 
   if (ut->ut_line[0] == '\0')
@@ -804 +819 @@
   while (user != NULL) 
     {
-      if (0 == sl_strncmp(user->ut_tty, ut->ut_line, UT_LINESIZE) ) 
+      if (0 == sl_strncmp((char*)(user->ut_tty), ut->ut_line, UT_LINESIZE) ) 
         break;
       userold = user;
@@ -827 +842 @@
           user = SH_ALLOC(sizeof(struct log_user));
           user->next       = userlist;
-          userlist         = user;
+          userlist         = (struct log_user *) user;
         }
-      (void) sl_strlcpy(user->ut_tty,  ut->ut_line, UT_LINESIZE+1);
-      (void) sl_strlcpy(user->name,    ut->ut_name, UT_NAMESIZE+1);
+      (void) sl_strlcpy((char*)(user->ut_tty),  ut->ut_line, UT_LINESIZE+1);
+      (void) sl_strlcpy((char*)(user->name),    ut->ut_name, UT_NAMESIZE+1);
 #ifdef HAVE_UTHOST
-      (void) sl_strlcpy(user->ut_host, ut->ut_host, UT_HOSTSIZE+1);
+      (void) sl_strlcpy((char*)(user->ut_host), ut->ut_host, UT_HOSTSIZE+1);
 #else
       user->ut_host[0] = '\0';
@@ -838 +853 @@
 #ifdef HAVE_UTADDR
       /*@-type@*//* ut_addr does exist !!! */
-      (void) sl_strlcpy(user->ut_ship, 
+      (void) sl_strlcpy((char*)(user->ut_ship), 
                         my_inet_ntoa(*(struct in_addr*)&(ut->ut_addr)), 16);
       /*@+type@*/
@@ -848 +863 @@
           || 0 == sl_strncmp(ut->ut_line, _("ttyq"), 4) /* in virt. console  */
           ) {
-        status = sh_utmp_login_a(user->name);
+        status = sh_utmp_login_a((char*)user->name);
         SH_MUTEX_LOCK(mutex_thread_nolog);
         (void) sh_unix_time (user->time, ttt, TIM_MAX);
@@ -872 +887 @@
         SH_MUTEX_UNLOCK(mutex_thread_nolog);
       } else
-        if (0 != sh_utmp_is_virtual(ut->ut_line, user->ut_host))
+        if (0 != sh_utmp_is_virtual(ut->ut_line, (char*)user->ut_host))
           {       
-            status = sh_utmp_login_a(user->name);
+            status = sh_utmp_login_a((char*)user->name);
             SH_MUTEX_LOCK(mutex_thread_nolog);
             (void) sh_unix_time (user->time, ttt, TIM_MAX);
@@ -911 +926 @@
       if (user != NULL)
         {
-          status = sh_utmp_login_r(user->name);
+          status = sh_utmp_login_r((char*)user->name);
           SH_MUTEX_LOCK(mutex_thread_nolog);
           (void) sh_unix_time (ut->ut_time, ttt, TIM_MAX);
@@ -937 +952 @@
           if (user == userlist)
             userlist = user->next;
-          sh_utmp_logout_morechecks(user);
-          SH_FREE(user);
+          sh_utmp_logout_morechecks((struct log_user *)user);
+          SH_FREE((struct log_user *)user);
           user = NULL;
         }