Created attachment 16023 [details] [review] Patch to correct it configure.in will #define DBUS_ENABLE_USERDB_CACHE if --disable-userdb-cache is not passed. However dbus/dbus-userdb{,-util}.c will only enable the code if DBUS_ENABLE_USER_CACHE is defined. So the userdb cache is currently broken.
Created attachment 16157 [details] [review] Patch in preferred format
Looks good, we may need to set disable to enable since by default we have had it off for some time. I'm bringing the discussion on the D-Bus list.
Possibly, though I'm running with it enabled now and not encountering any new or strange problems. If it's left disabled, #15589 will need to be attended to with some kind of refcounting for the objects
Never mind, that was after 1.0 so it is ok to fix as it.
*** Bug 15455 has been marked as a duplicate of this bug. ***
I'm pretty sure the bus is outrageously slow with this disabled, too (at least if you have any policy rules based on group membership)
commit 80306140b9c91bce319f50a55694f205a9a4392f Author: Colin Walters <walters@verbum.org> Date: Fri May 30 11:14:46 2008 -0400 Bug 15588: Fix typo in #ifdef for userdb cache (Scott James Remnant) * dbus/dbus-userdb-util.c, dbus/dbus-userdb.c: Correct name of macro used in #ifdef block to match that defined by configure, otherwise the userdb cache will never be enabled.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.