Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(793)

Side by Side Diff: gdbus/watch.c

Issue 3137021: fix memory leak in flimflam dbus support (Closed) Base URL: ssh://git@chromiumos-git//flimflam.git
Patch Set: Created 10 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * 2 *
3 * D-Bus helper library 3 * D-Bus helper library
4 * 4 *
5 * Copyright (C) 2004-2009 Marcel Holtmann <marcel@holtmann.org> 5 * Copyright (C) 2004-2009 Marcel Holtmann <marcel@holtmann.org>
6 * 6 *
7 * 7 *
8 * This program is free software; you can redistribute it and/or modify 8 * This program is free software; you can redistribute it and/or modify
9 * it under the terms of the GNU General Public License as published by 9 * it under the terms of the GNU General Public License as published by
10 * the Free Software Foundation; either version 2 of the License, or 10 * the Free Software Foundation; either version 2 of the License, or
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
379 g_free(data); 379 g_free(data);
380 goto done; 380 goto done;
381 } 381 }
382 382
383 if (call == NULL) { 383 if (call == NULL) {
384 error("D-Bus connection not available"); 384 error("D-Bus connection not available");
385 g_free(data); 385 g_free(data);
386 goto done; 386 goto done;
387 } 387 }
388 388
389 » dbus_pending_call_set_notify(call, service_reply, data, NULL); 389 » dbus_pending_call_set_notify(call, service_reply, data, g_free);
390 390
391 done: 391 done:
392 dbus_message_unref(message); 392 dbus_message_unref(message);
393 } 393 }
394 394
395 guint g_dbus_add_service_watch(DBusConnection *connection, const char *name, 395 guint g_dbus_add_service_watch(DBusConnection *connection, const char *name,
396 GDBusWatchFunction connect, 396 GDBusWatchFunction connect,
397 GDBusWatchFunction disconnect, 397 GDBusWatchFunction disconnect,
398 void *user_data, GDBusDestroyFunction destroy) 398 void *user_data, GDBusDestroyFunction destroy)
399 { 399 {
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
502 { 502 {
503 struct name_data *data; 503 struct name_data *data;
504 504
505 while ((data = name_data_find(connection, NULL))) { 505 while ((data = name_data_find(connection, NULL))) {
506 name_listeners = g_slist_remove(name_listeners, data); 506 name_listeners = g_slist_remove(name_listeners, data);
507 name_data_call_and_free(data); 507 name_data_call_and_free(data);
508 } 508 }
509 509
510 dbus_connection_remove_filter(connection, name_exit_filter, NULL); 510 dbus_connection_remove_filter(connection, name_exit_filter, NULL);
511 } 511 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698