The 'on_name_owner_changed' function is a little bit weird.  I'll paste the starting part:

----

{
  GDBusProxy *proxy = G_DBUS_PROXY (user_data);
  const gchar *name;
  const gchar *old_owner;
  const gchar *new_owner;

  /* if we are already trying to load properties, cancel that */
  if (proxy->priv->get_all_cancellable != NULL)
    {
      g_cancellable_cancel (proxy->priv->get_all_cancellable);
      proxy->priv->get_all_cancellable = NULL;
    }

  g_variant_get (parameters,
                 "(&s&s&s)",
                 &name,
                 &old_owner,
                 &new_owner);

  /* we only care about a specific name */
  if (g_strcmp0 (name, proxy->priv->name) != 0)
    goto out;

----


Why are we cancelling the pending property query before we bail out due to having the incorrect name?  That really looks like a bug.

Of course, I think the real issue is probably that there's no bug at all but rather it's impossible for the strcmp to fail (since when we register for the signal we already specify the arg0 as part of the match rules).  In that case probably we should remove it entirely.

Either way, something should be changed.