Mercurial > pidgin.yaz
view libpurple/plugins/ipc-test-server.c @ 22393:dfe4a3e3336a
serv_got_joined_chat() emits "chat-joined" but it also calls
purple_conversation_new() which emits "conversation-created", so attaching
to both signals would make create_conv and attach_signals to be called twice,
leaking the list of signal ids:
==30871== 67 (43 direct, 24 indirect) bytes in 3 blocks are definitely lost in loss record 94 of 265
==30871== at 0x4006A6E: malloc (vg_replace_malloc.c:207)
==30871== by 0xA1CAF5: g_malloc (in /lib/libglib-2.0.so.0.1400.6)
==30871== by 0xA31B3A: g_slice_alloc (in /lib/libglib-2.0.so.0.1400.6)
==30871== by 0xA32D9A: g_slist_append (in /lib/libglib-2.0.so.0.1400.6)
==30871== by 0x4F1CA4A: attach_signals (notify.c:321)
==30871== by 0x4F1CDCA: conv_created (notify.c:388)
Credit goes to Andrew Gaul for sending me this valgrind trace, blame me if
this breaks anything.
author | Ka-Hing Cheung <khc@hxbc.us> |
---|---|
date | Mon, 03 Mar 2008 07:03:54 +0000 |
parents | 3cc856ca2338 |
children |
line wrap: on
line source
/* * IPC test server plugin. * * Copyright (C) 2003 Christian Hammond. * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License as * published by the Free Software Foundation; either version 2 of the * License, or (at your option) any later version. * * This program is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * General Public License for more details. * * You should have received a copy of the GNU General Public License * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA * 02111-1301, USA. */ #define IPC_TEST_SERVER_PLUGIN_ID "core-ipc-test-server" #include "internal.h" #include "debug.h" #include "plugin.h" #include "version.h" static int add_func(int i1, int i2) { purple_debug_misc("ipc-test-server", "Got %d, %d, returning %d\n", i1, i2, i1 + i2); return i1 + i2; } static int sub_func(int i1, int i2) { purple_debug_misc("ipc-test-server", "Got %d, %d, returning %d\n", i1, i2, i1 - i2); return i1 - i2; } static gboolean plugin_load(PurplePlugin *plugin) { purple_plugin_ipc_register(plugin, "add", PURPLE_CALLBACK(add_func), purple_marshal_INT__INT_INT, purple_value_new(PURPLE_TYPE_INT), 2, purple_value_new(PURPLE_TYPE_INT), purple_value_new(PURPLE_TYPE_INT)); purple_plugin_ipc_register(plugin, "sub", PURPLE_CALLBACK(sub_func), purple_marshal_INT__INT_INT, purple_value_new(PURPLE_TYPE_INT), 2, purple_value_new(PURPLE_TYPE_INT), purple_value_new(PURPLE_TYPE_INT)); return TRUE; } static PurplePluginInfo info = { PURPLE_PLUGIN_MAGIC, PURPLE_MAJOR_VERSION, PURPLE_MINOR_VERSION, PURPLE_PLUGIN_STANDARD, /**< type */ NULL, /**< ui_requirement */ 0, /**< flags */ NULL, /**< dependencies */ PURPLE_PRIORITY_DEFAULT, /**< priority */ IPC_TEST_SERVER_PLUGIN_ID, /**< id */ N_("IPC Test Server"), /**< name */ DISPLAY_VERSION, /**< version */ /** summary */ N_("Test plugin IPC support, as a server."), /** description */ N_("Test plugin IPC support, as a server. This registers the IPC " "commands."), "Christian Hammond <chipx86@gnupdate.org>", /**< author */ PURPLE_WEBSITE, /**< homepage */ plugin_load, /**< load */ NULL, /**< unload */ NULL, /**< destroy */ NULL, /**< ui_info */ NULL, /**< extra_info */ NULL, NULL }; static void init_plugin(PurplePlugin *plugin) { } PURPLE_INIT_PLUGIN(ipctestserver, init_plugin, info)