DBusTransport: do not assert that autolaunch address is non-empty
authorSimon McVittie <simon.mcvittie@collabora.co.uk>
Tue, 3 Jul 2012 08:26:27 +0000 (09:26 +0100)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Tue, 3 Jul 2012 18:02:53 +0000 (19:02 +0100)
dbus-launch can apparently return an empty address under certain
circumstances, and dbus_parse_address() in the next line will return
a nice DBusError for an empty address rather than aborting the process.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=51657
Bug-Debian: http://bugs.debian.org/680027
Reviewed-by: David Zeuthen <davidz@redhat.com>
dbus/dbus-transport.c

index ba720f2..6b58fda 100644 (file)
@@ -253,7 +253,6 @@ check_address (const char *address, DBusError *error)
   int len, i;
 
   _dbus_assert (address != NULL);
-  _dbus_assert (*address != '\0');
 
   if (!dbus_parse_address (address, &entries, &len, error))
     return NULL;              /* not a valid address */