[Patch] Fix _dbus_group_info_free_allocated to not call itself recursively forever

John (J5) Palmieri johnp at redhat.com
Thu Feb 23 08:35:20 PST 2006


_dbus_user_info_free is in dbus-sysdeps.c so ok'ing this patch.  Feel
free to commit.

On Thu, 2006-02-23 at 10:11 -0500, John (J5) Palmieri wrote:
> Hmm, I wonder if this fixes a bug where dbus hangs on startup when one
> of our users has ldap enabled in /etc/nsswitch.conf.
> 
> The code looks like it is doing the right thing.  Question though, why
> does dbus_group_info_free need to move to dbus/dbus-sysdeps.c?  It
> should only be used in the daemon right?
> 
> On Thu, 2006-02-23 at 15:11 +0100, Sjoerd Simons wrote:
> > Hi,
> > 
> >   _dbus_group_info_free_allocated currently just calls itself again (without
> >   doing anything), leading to an infinite loop. Attached patch lets it call
> >   _dbus_group_info_free, making it act as stated in the doxygen
> >   documentation.
> > 
> >   To accomplish this _dbus_group_info_free had to be moved from
> >   dbus/dbus-sysdeps-util.c to dbus/dbus-sysdeps.c (also in attached patch)
> > 
> >   This bug is triggerd when there is an unknown group mentioned in a bus
> >   configuration file.
> > 
> >   Sjoerd
> > _______________________________________________
> > dbus mailing list
> > dbus at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dbus

-- 
John (J5) Palmieri <johnp at redhat.com>



More information about the dbus mailing list