the API contract says sitter_p can be NULL, so let's check it (FDO Bug #12919)

2008-01-15  John (J5) Palmieri  <johnp@redhat.com>

	* patch by Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>

	* dbus/dbus-spawn.c (_dbus_spawn_async_with_babysitter): the API
	contract says sitter_p can be NULL, so let's check it (FDO Bug #12919)

diff --git a/ChangeLog b/ChangeLog
index d0020c7..dfa00b1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,13 @@
        
        * patch by Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>
 
+       * dbus/dbus-spawn.c (_dbus_spawn_async_with_babysitter): the API 
+       contract says sitter_p can be NULL, so let's check it (FDO Bug #12919)
+
+2008-01-15  John (J5) Palmieri  <johnp@redhat.com>
+       
+       * patch by Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>
+
        * dbus/dbus-spawn.c (read_ints, read_pid): use correct ssize_t type
        instead of size_t (FDO Bug #12862)
 

diff --git a/dbus/dbus-spawn.c b/dbus/dbus-spawn.c
index 52d3eef..ba6e6dd 100644 (file)
--- a/dbus/dbus-spawn.c
+++ b/dbus/dbus-spawn.c
@@ -1085,7 +1085,9 @@ _dbus_spawn_async_with_babysitter (DBusBabysitter          **sitter_p,
   
   _DBUS_ASSERT_ERROR_IS_CLEAR (error);
 
-  *sitter_p = NULL;
+  if (sitter_p != NULL)
+    *sitter_p = NULL;
+
   sitter = NULL;
 
   sitter = _dbus_babysitter_new ();