# HG changeset patch # User Mark Doliner # Date 1238146598 0 # Node ID f4161d588b2511e4a77c439f730932c254aaf017 # Parent 58daeb191c102e055dc8b17d7329c098f8f2ef3d Fix a potential memleak by freeing this regardless of whether hasActions is set diff -r 58daeb191c10 -r f4161d588b25 libpurple/protocols/jabber/xdata.c --- a/libpurple/protocols/jabber/xdata.c Fri Mar 27 09:34:27 2009 +0000 +++ b/libpurple/protocols/jabber/xdata.c Fri Mar 27 09:36:38 2009 +0000 @@ -153,11 +153,12 @@ } g_free(data); - if (hasActions) { + if (hasActions) cb(js, result, actionhandle, user_data); - g_free(actionhandle); - } else + else ((jabber_x_data_cb)cb)(js, result, user_data); + + g_free(actionhandle); } static void jabber_x_data_cancel_cb(struct jabber_x_data_data *data, PurpleRequestFields *fields) {