projects
/
platform
/
upstream
/
glib.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
GSettings: fix check for delaying backend subscription
[platform/upstream/glib.git]
/
gio
/
gdbusobjectproxy.c
diff --git
a/gio/gdbusobjectproxy.c
b/gio/gdbusobjectproxy.c
index
46f249b
..
9788925
100644
(file)
--- a/
gio/gdbusobjectproxy.c
+++ b/
gio/gdbusobjectproxy.c
@@
-13,9
+13,7
@@
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General
- * Public License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place, Suite 330,
- * Boston, MA 02111-1307, USA.
+ * Public License along with this library; if not, see <http://www.gnu.org/licenses/>.
*
* Author: David Zeuthen <davidz@redhat.com>
*/
*
* Author: David Zeuthen <davidz@redhat.com>
*/
@@
-46,6
+44,7
@@
struct _GDBusObjectProxyPrivate
{
struct _GDBusObjectProxyPrivate
{
+ GMutex lock;
GHashTable *map_name_to_iface;
gchar *object_path;
GDBusConnection *connection;
GHashTable *map_name_to_iface;
gchar *object_path;
GDBusConnection *connection;
@@
-54,14
+53,15
@@
struct _GDBusObjectProxyPrivate
enum
{
PROP_0,
enum
{
PROP_0,
- PROP_OBJECT_PATH,
- PROP_CONNECTION
+ PROP_
G_
OBJECT_PATH,
+ PROP_
G_
CONNECTION
};
static void dbus_object_interface_init (GDBusObjectIface *iface);
G_DEFINE_TYPE_WITH_CODE (GDBusObjectProxy, g_dbus_object_proxy, G_TYPE_OBJECT,
};
static void dbus_object_interface_init (GDBusObjectIface *iface);
G_DEFINE_TYPE_WITH_CODE (GDBusObjectProxy, g_dbus_object_proxy, G_TYPE_OBJECT,
- G_IMPLEMENT_INTERFACE (G_TYPE_DBUS_OBJECT, dbus_object_interface_init));
+ G_ADD_PRIVATE (GDBusObjectProxy)
+ G_IMPLEMENT_INTERFACE (G_TYPE_DBUS_OBJECT, dbus_object_interface_init))
static void
g_dbus_object_proxy_finalize (GObject *object)
static void
g_dbus_object_proxy_finalize (GObject *object)
@@
-70,6
+70,12
@@
g_dbus_object_proxy_finalize (GObject *object)
g_hash_table_unref (proxy->priv->map_name_to_iface);
g_hash_table_unref (proxy->priv->map_name_to_iface);
+ g_clear_object (&proxy->priv->connection);
+
+ g_free (proxy->priv->object_path);
+
+ g_mutex_clear (&proxy->priv->lock);
+
if (G_OBJECT_CLASS (g_dbus_object_proxy_parent_class)->finalize != NULL)
G_OBJECT_CLASS (g_dbus_object_proxy_parent_class)->finalize (object);
}
if (G_OBJECT_CLASS (g_dbus_object_proxy_parent_class)->finalize != NULL)
G_OBJECT_CLASS (g_dbus_object_proxy_parent_class)->finalize (object);
}
@@
-84,16
+90,18
@@
g_dbus_object_proxy_get_property (GObject *object,
switch (prop_id)
{
switch (prop_id)
{
- case PROP_OBJECT_PATH:
+ case PROP_G_OBJECT_PATH:
+ g_mutex_lock (&proxy->priv->lock);
g_value_set_string (value, proxy->priv->object_path);
g_value_set_string (value, proxy->priv->object_path);
+ g_mutex_unlock (&proxy->priv->lock);
break;
break;
- case PROP_CONNECTION:
+ case PROP_
G_
CONNECTION:
g_value_set_object (value, g_dbus_object_proxy_get_connection (proxy));
break;
default:
g_value_set_object (value, g_dbus_object_proxy_get_connection (proxy));
break;
default:
- G_OBJECT_WARN_INVALID_PROPERTY_ID (
_object
, prop_id, pspec);
+ G_OBJECT_WARN_INVALID_PROPERTY_ID (
proxy
, prop_id, pspec);
break;
}
}
break;
}
}
@@
-108,16
+116,20
@@
g_dbus_object_proxy_set_property (GObject *object,
switch (prop_id)
{
switch (prop_id)
{
- case PROP_OBJECT_PATH:
+ case PROP_G_OBJECT_PATH:
+ g_mutex_lock (&proxy->priv->lock);
proxy->priv->object_path = g_value_dup_string (value);
proxy->priv->object_path = g_value_dup_string (value);
+ g_mutex_unlock (&proxy->priv->lock);
break;
break;
- case PROP_CONNECTION:
+ case PROP_G_CONNECTION:
+ g_mutex_lock (&proxy->priv->lock);
proxy->priv->connection = g_value_dup_object (value);
proxy->priv->connection = g_value_dup_object (value);
+ g_mutex_unlock (&proxy->priv->lock);
break;
default:
break;
default:
- G_OBJECT_WARN_INVALID_PROPERTY_ID (
_object
, prop_id, pspec);
+ G_OBJECT_WARN_INVALID_PROPERTY_ID (
proxy
, prop_id, pspec);
break;
}
}
break;
}
}
@@
-132,15
+144,15
@@
g_dbus_object_proxy_class_init (GDBusObjectProxyClass *klass)
gobject_class->get_property = g_dbus_object_proxy_get_property;
/**
gobject_class->get_property = g_dbus_object_proxy_get_property;
/**
- * GDBusObjectProxy:object-path:
+ * GDBusObjectProxy:
g-
object-path:
*
* The object path of the proxy.
*
* Since: 2.30
*/
g_object_class_install_property (gobject_class,
*
* The object path of the proxy.
*
* Since: 2.30
*/
g_object_class_install_property (gobject_class,
- PROP_OBJECT_PATH,
- g_param_spec_string ("object-path",
+ PROP_
G_
OBJECT_PATH,
+ g_param_spec_string ("
g-
object-path",
"Object Path",
"The object path of the proxy",
NULL,
"Object Path",
"The object path of the proxy",
NULL,
@@
-149,31
+161,28
@@
g_dbus_object_proxy_class_init (GDBusObjectProxyClass *klass)
G_PARAM_STATIC_STRINGS));
/**
G_PARAM_STATIC_STRINGS));
/**
- * GDBusObjectProxy:connection:
+ * GDBusObjectProxy:
g-
connection:
*
* The connection of the proxy.
*
* Since: 2.30
*/
g_object_class_install_property (gobject_class,
*
* The connection of the proxy.
*
* Since: 2.30
*/
g_object_class_install_property (gobject_class,
- PROP_CONNECTION,
- g_param_spec_object ("connection",
+ PROP_
G_
CONNECTION,
+ g_param_spec_object ("
g-
connection",
"Connection",
"The connection of the proxy",
G_TYPE_DBUS_CONNECTION,
G_PARAM_READWRITE |
G_PARAM_CONSTRUCT_ONLY |
G_PARAM_STATIC_STRINGS));
"Connection",
"The connection of the proxy",
G_TYPE_DBUS_CONNECTION,
G_PARAM_READWRITE |
G_PARAM_CONSTRUCT_ONLY |
G_PARAM_STATIC_STRINGS));
-
- g_type_class_add_private (klass, sizeof (GDBusObjectProxyPrivate));
}
static void
g_dbus_object_proxy_init (GDBusObjectProxy *proxy)
{
}
static void
g_dbus_object_proxy_init (GDBusObjectProxy *proxy)
{
- proxy->priv = G_TYPE_INSTANCE_GET_PRIVATE (proxy,
- G_TYPE_DBUS_OBJECT_PROXY,
- GDBusObjectProxyPrivate);
+ proxy->priv = g_dbus_object_proxy_get_instance_private (proxy);
+ g_mutex_init (&proxy->priv->lock);
proxy->priv->map_name_to_iface = g_hash_table_new_full (g_str_hash,
g_str_equal,
g_free,
proxy->priv->map_name_to_iface = g_hash_table_new_full (g_str_hash,
g_str_equal,
g_free,
@@
-184,12
+193,16
@@
static const gchar *
g_dbus_object_proxy_get_object_path (GDBusObject *object)
{
GDBusObjectProxy *proxy = G_DBUS_OBJECT_PROXY (object);
g_dbus_object_proxy_get_object_path (GDBusObject *object)
{
GDBusObjectProxy *proxy = G_DBUS_OBJECT_PROXY (object);
- return proxy->priv->object_path;
+ const gchar *ret;
+ g_mutex_lock (&proxy->priv->lock);
+ ret = proxy->priv->object_path;
+ g_mutex_unlock (&proxy->priv->lock);
+ return ret;
}
/**
* g_dbus_object_proxy_get_connection:
}
/**
* g_dbus_object_proxy_get_connection:
- * @proxy:
A #GDBusObjectProxy.
+ * @proxy:
a #GDBusObjectProxy
*
* Gets the connection that @proxy is for.
*
*
* Gets the connection that @proxy is for.
*
@@
-201,8
+214,12
@@
g_dbus_object_proxy_get_object_path (GDBusObject *object)
GDBusConnection *
g_dbus_object_proxy_get_connection (GDBusObjectProxy *proxy)
{
GDBusConnection *
g_dbus_object_proxy_get_connection (GDBusObjectProxy *proxy)
{
+ GDBusConnection *ret;
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
- return proxy->priv->connection;
+ g_mutex_lock (&proxy->priv->lock);
+ ret = proxy->priv->connection;
+ g_mutex_unlock (&proxy->priv->lock);
+ return ret;
}
static GDBusInterface *
}
static GDBusInterface *
@@
-215,9
+232,12
@@
g_dbus_object_proxy_get_interface (GDBusObject *object,
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
g_return_val_if_fail (g_dbus_is_interface_name (interface_name), NULL);
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
g_return_val_if_fail (g_dbus_is_interface_name (interface_name), NULL);
+ g_mutex_lock (&proxy->priv->lock);
ret = g_hash_table_lookup (proxy->priv->map_name_to_iface, interface_name);
if (ret != NULL)
g_object_ref (ret);
ret = g_hash_table_lookup (proxy->priv->map_name_to_iface, interface_name);
if (ret != NULL)
g_object_ref (ret);
+ g_mutex_unlock (&proxy->priv->lock);
+
return (GDBusInterface *) ret; /* TODO: proper cast */
}
return (GDBusInterface *) ret; /* TODO: proper cast */
}
@@
-226,31
+246,42
@@
g_dbus_object_proxy_get_interfaces (GDBusObject *object)
{
GDBusObjectProxy *proxy = G_DBUS_OBJECT_PROXY (object);
GList *ret;
{
GDBusObjectProxy *proxy = G_DBUS_OBJECT_PROXY (object);
GList *ret;
- GHashTableIter iter;
- GDBusProxy *interface_proxy;
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
ret = NULL;
g_return_val_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy), NULL);
ret = NULL;
- g_hash_table_iter_init (&iter, proxy->priv->map_name_to_iface);
- while (g_hash_table_iter_next (&iter, NULL, (gpointer) &interface_proxy))
- ret = g_list_prepend (ret, g_object_ref (interface_proxy));
+ g_mutex_lock (&proxy->priv->lock);
+ ret = g_hash_table_get_values (proxy->priv->map_name_to_iface);
+ g_list_foreach (ret, (GFunc) g_object_ref, NULL);
+ g_mutex_unlock (&proxy->priv->lock);
return ret;
}
/* ---------------------------------------------------------------------------------------------------- */
return ret;
}
/* ---------------------------------------------------------------------------------------------------- */
+/**
+ * g_dbus_object_proxy_new:
+ * @connection: a #GDBusConnection
+ * @object_path: the object path
+ *
+ * Creates a new #GDBusObjectProxy for the given connection and
+ * object path.
+ *
+ * Returns: a new #GDBusObjectProxy
+ *
+ * Since: 2.30
+ */
GDBusObjectProxy *
g_dbus_object_proxy_new (GDBusConnection *connection,
GDBusObjectProxy *
g_dbus_object_proxy_new (GDBusConnection *connection,
- const gchar *object_path)
+ const gchar
*object_path)
{
g_return_val_if_fail (G_IS_DBUS_CONNECTION (connection), NULL);
g_return_val_if_fail (g_variant_is_object_path (object_path), NULL);
return G_DBUS_OBJECT_PROXY (g_object_new (G_TYPE_DBUS_OBJECT_PROXY,
{
g_return_val_if_fail (G_IS_DBUS_CONNECTION (connection), NULL);
g_return_val_if_fail (g_variant_is_object_path (object_path), NULL);
return G_DBUS_OBJECT_PROXY (g_object_new (G_TYPE_DBUS_OBJECT_PROXY,
- "object-path", object_path,
- "connection", connection,
+ "
g-
object-path", object_path,
+ "
g-
connection", connection,
NULL));
}
NULL));
}
@@
-259,16
+290,35
@@
_g_dbus_object_proxy_add_interface (GDBusObjectProxy *proxy,
GDBusProxy *interface_proxy)
{
const gchar *interface_name;
GDBusProxy *interface_proxy)
{
const gchar *interface_name;
+ GDBusProxy *interface_proxy_to_remove;
g_return_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy));
g_return_if_fail (G_IS_DBUS_PROXY (interface_proxy));
g_return_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy));
g_return_if_fail (G_IS_DBUS_PROXY (interface_proxy));
+ g_mutex_lock (&proxy->priv->lock);
+
interface_name = g_dbus_proxy_get_interface_name (interface_proxy);
interface_name = g_dbus_proxy_get_interface_name (interface_proxy);
- _g_dbus_object_proxy_remove_interface (proxy, interface_name);
+ interface_proxy_to_remove = g_hash_table_lookup (proxy->priv->map_name_to_iface, interface_name);
+ if (interface_proxy_to_remove != NULL)
+ {
+ g_object_ref (interface_proxy_to_remove);
+ g_warn_if_fail (g_hash_table_remove (proxy->priv->map_name_to_iface, interface_name));
+ }
g_hash_table_insert (proxy->priv->map_name_to_iface,
g_strdup (interface_name),
g_object_ref (interface_proxy));
g_hash_table_insert (proxy->priv->map_name_to_iface,
g_strdup (interface_name),
g_object_ref (interface_proxy));
+ g_object_ref (interface_proxy);
+
+ g_mutex_unlock (&proxy->priv->lock);
+
+ if (interface_proxy_to_remove != NULL)
+ {
+ g_signal_emit_by_name (proxy, "interface-removed", interface_proxy_to_remove);
+ g_object_unref (interface_proxy_to_remove);
+ }
+
g_signal_emit_by_name (proxy, "interface-added", interface_proxy);
g_signal_emit_by_name (proxy, "interface-added", interface_proxy);
+ g_object_unref (interface_proxy);
}
void
}
void
@@
-280,14
+330,21
@@
_g_dbus_object_proxy_remove_interface (GDBusObjectProxy *proxy,
g_return_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy));
g_return_if_fail (g_dbus_is_interface_name (interface_name));
g_return_if_fail (G_IS_DBUS_OBJECT_PROXY (proxy));
g_return_if_fail (g_dbus_is_interface_name (interface_name));
+ g_mutex_lock (&proxy->priv->lock);
+
interface_proxy = g_hash_table_lookup (proxy->priv->map_name_to_iface, interface_name);
if (interface_proxy != NULL)
{
g_object_ref (interface_proxy);
g_warn_if_fail (g_hash_table_remove (proxy->priv->map_name_to_iface, interface_name));
interface_proxy = g_hash_table_lookup (proxy->priv->map_name_to_iface, interface_name);
if (interface_proxy != NULL)
{
g_object_ref (interface_proxy);
g_warn_if_fail (g_hash_table_remove (proxy->priv->map_name_to_iface, interface_name));
+ g_mutex_unlock (&proxy->priv->lock);
g_signal_emit_by_name (proxy, "interface-removed", interface_proxy);
g_object_unref (interface_proxy);
}
g_signal_emit_by_name (proxy, "interface-removed", interface_proxy);
g_object_unref (interface_proxy);
}
+ else
+ {
+ g_mutex_unlock (&proxy->priv->lock);
+ }
}
static void
}
static void