Launchpad has imported 4 comments from the remote bug at
https://bugs.freedesktop.org/show_bug.cgi?id=65130.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2013-05-29T13:58:38+00:00 Tbzatek wrote:

Created attachment 79966
[PATCH] Use GOnce for interface type registration

In certain cases, randomly, udisksd was failing with the following
messages:

> (udisksd:10020): GLib-GObject-WARNING **: cannot register existing type 
> `PolkitSubject'
> (udisksd:10020): GLib-GObject-CRITICAL **: g_type_interface_add_prerequisite: 
> assertion `G_TYPE_IS_INTERFACE (interface_type)' failed
> (udisksd:10020): GLib-GObject-WARNING **: invalid cast from 
> `PolkitSystemBusName' to `<invalid>'
> (udisksd:10020): GLib-GObject-WARNING **: cannot register existing type 
> `PolkitSubject'
> (udisksd:10020): GLib-GObject-WARNING **: cannot register existing type 
> `PolkitSubject'
> (udisksd:10020): GLib-GObject-CRITICAL **: g_type_interface_add_prerequisite: 
> assertion `G_TYPE_IS_INTERFACE (interface_type)' failed
> (udisksd:10020): GLib-GObject-CRITICAL **: g_type_interface_add_prerequisite: 
> assertion `G_TYPE_IS_INTERFACE (interface_type)' failed
> ** (udisksd:10020): CRITICAL **: polkit_authority_check_authorization_sync: 
> assertion `POLKIT_IS_SUBJECT (subject)' failed
> ** (udisksd:10020): CRITICAL **: polkit_authority_check_authorization_sync: 
> assertion `POLKIT_IS_SUBJECT (subject)' failed

Further debugging showed missing locking in polkit_subject_get_type()
and potential race explaining the first GObject warning.

The attached patch makes the type registration use modern conventions
with GOnce.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/policykit-1/+bug/1236510/comments/0

------------------------------------------------------------------------
On 2013-05-29T14:14:34+00:00 Colin Walters wrote:

Looks good.  I amended the patch to include a link back to this bug.

http://cgit.freedesktop.org/polkit/commit/?id=20ad116a6582e57d20f9d8197758947918753a4c

Reply at:
https://bugs.launchpad.net/ubuntu/+source/policykit-1/+bug/1236510/comments/1

------------------------------------------------------------------------
On 2013-05-29T14:34:40+00:00 Hans de Goede wrote:

I can confirm that this patch fixes my reproducable udisksd crashes.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/policykit-1/+bug/1236510/comments/2

------------------------------------------------------------------------
On 2013-05-29T19:23:34+00:00 Zeuthen wrote:

Nice catch guys, thanks for fixing this!

Reply at:
https://bugs.launchpad.net/ubuntu/+source/policykit-1/+bug/1236510/comments/3


** Changed in: policykit
       Status: Unknown => Fix Released

** Changed in: policykit
   Importance: Unknown => High

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1236510

Title:
  udisksd crashed with SIGSEGV in
  udisks_daemon_util_check_authorization_sync()

To manage notifications about this bug go to:
https://bugs.launchpad.net/policykit/+bug/1236510/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to