summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Clasen <mclasen@redhat.com>2011-10-19 20:27:55 -0400
committerRyan Lortie <desrt@desrt.ca>2011-10-31 19:18:08 -0400
commita271dacba8eb0057455ed1d51304a983279b67e2 (patch)
tree00257fb8cddcce447c3d6399cf21925ba8e35950
parent4f1d6f12926d6c5512d0f127720b8e307c454b41 (diff)
downloaddconf-a271dacba8eb0057455ed1d51304a983279b67e2.tar.gz
Drop a dead variable
The GError in add_match_done is never used. Whats worse, it is dereferenced in the error path. https://bugzilla.gnome.org/show_bug.cgi?id=662259
-rw-r--r--dbus-1/dconf-dbus-1.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/dbus-1/dconf-dbus-1.c b/dbus-1/dconf-dbus-1.c
index 421ea12..abf529d 100644
--- a/dbus-1/dconf-dbus-1.c
+++ b/dbus-1/dconf-dbus-1.c
@@ -534,7 +534,6 @@ add_match_done (DBusPendingCall *pending,
gpointer user_data)
{
Watch *watch = user_data;
- GError *error = NULL;
GVariant *reply;
reply = dconf_dbus_client_send_finish (pending);
@@ -547,9 +546,7 @@ add_match_done (DBusPendingCall *pending,
* watch structure in the list).
*/
- g_critical ("DBus AddMatch for dconf path '%s': %s",
- watch->name, error->message);
- g_error_free (error);
+ g_critical ("DBus AddMatch for dconf path '%s'", watch->name);
watch_unref (watch);
return;