Bug#694340: libgconf2-4: libgconf built with the dbus backend requires dbus-glib to be initialized with threading

[Josselin Mouette]

severity 694340 serious
thanks

Hi,

Le lundi 26 novembre 2012 à 01:36 +0800, Chow Loong Jin a écrit :
> 
> +  /* Initialize DBus Glib for multithreading -- this fixes race conditions when
> +     multi-threaded applications use gconf. Additionally, although the API
> +     documentation says that dbus_g_thread_init() may only be called once, it is
> +     actually really a wrapper for dbus_threads_init_default() which can be
> +     called as many times as necessary. */ 

Thanks for the patch, but I’m not fond of violating the API just because
it works for the moment. This is precisely the kind of code that can
trigger later the kind of bug you are trying to fix.

OTOH the risk is low, since dbus-glib is a dying library.

Maybe we should just port GConf to GDBus instead, which doesn’t have
this issue, but this is not a small change.

Cheers,
-- 
 .''`.      Josselin Mouette
: :' :
`. `'
  `-