2004-08-28 Havoc Pennington <hp@redhat.com>

	* dbus/dbus-userdb.c (_dbus_is_console_user): remove unused variable

        More fixes from Steve Grubb

	* dbus/dbus-sysdeps.c (_dbus_connect_tcp_socket): fix fd leak
	(_dbus_listen_tcp_socket): fix fd leak

	* dbus/dbus-spawn.c (read_pid, read_ints): move the "again:" for
	EINTR to a bit lower in the code

diff --git a/ChangeLog b/ChangeLog
index ba29548bce107166e7062da30c364dcff0681368..e47a0f31e27e15a5b027c8a05d6b3e5d0e34516e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2004-08-28  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-userdb.c (_dbus_is_console_user): remove unused variable
+
+        More fixes from Steve Grubb
+       
+       * dbus/dbus-sysdeps.c (_dbus_connect_tcp_socket): fix fd leak
+       (_dbus_listen_tcp_socket): fix fd leak
+
+       * dbus/dbus-spawn.c (read_pid, read_ints): move the "again:" for
+       EINTR to a bit lower in the code
+
 2004-08-26  Jon Trowbridge  <trow@ximian.com>
 
        * bus/driver.c (bus_driver_handle_service_exists): Respond with

diff --git a/dbus/dbus-spawn.c b/dbus/dbus-spawn.c
index a70083037afbfdd70a2ac81ee9a5accca499cab9..03947cce417f3db6b64ef4062019187b1ef0c0d2 100644 (file)
--- a/dbus/dbus-spawn.c
+++ b/dbus/dbus-spawn.c
@@ -72,13 +72,13 @@ read_ints (int        fd,
     {
       size_t chunk;
       size_t to_read;
-      
-    again:
 
       to_read = sizeof (int) * n_ints_in_buf - bytes;
 
       if (to_read == 0)
         break;
+
+    again:
       
       chunk = read (fd,
                     ((char*)buf) + bytes,
@@ -128,11 +128,12 @@ read_pid (int        fd,
       size_t chunk;    
       size_t to_read;
       
-    again:
       to_read = sizeof (pid_t) - bytes;
 
       if (to_read == 0)
         break;
+
+    again:
       
       chunk = read (fd,
                     ((char*)buf) + bytes,

diff --git a/dbus/dbus-sysdeps.c b/dbus/dbus-sysdeps.c
index 4af70931a6d15babc0d90b34382434eb59b2dcec..629adb6b6b5fdeb0f1f71f29061bc7beabb99e6e 100644 (file)
--- a/dbus/dbus-sysdeps.c
+++ b/dbus/dbus-sysdeps.c
@@ -622,6 +622,7 @@ _dbus_connect_tcp_socket (const char     *host,
                       _dbus_error_from_errno (errno),
                       "Failed to lookup hostname: %s",
                       host);
+      close (fd);
       return -1;
     }
   
@@ -695,6 +696,7 @@ _dbus_listen_tcp_socket (const char     *host,
                       _dbus_error_from_errno (errno),
                       "Failed to lookup hostname: %s",
                       host);
+      close (listen_fd);
       return -1;
     }
   

diff --git a/dbus/dbus-userdb.c b/dbus/dbus-userdb.c
index 833256c56f812227cd756a3f6250701c6867eb08..d8477cd2adbc6c224491703301d6d92a6c85773b 100644 (file)
--- a/dbus/dbus-userdb.c
+++ b/dbus/dbus-userdb.c
@@ -413,7 +413,6 @@ _dbus_is_console_user (dbus_uid_t uid,
 
   DBusUserDatabase *db;
   const DBusUserInfo *info;
-  DBusString *console_file;
   dbus_bool_t result = FALSE; 
 
   _dbus_user_database_lock_system ();