comparison libpurple/protocols/msn/notification.c @ 28353:6a23d7f84143

Allow setting a destroy-function for the data. Fixes a leak.

28352:3061a0c72b26

	data->cb = cb;
	data->data = cb_data;

	trans = msn_transaction_new(cmdproc, "FQY", "%d", payload_len);
	msn_transaction_set_payload(trans, payload, payload_len);
	msn_transaction_set_data(trans, data);	/* XXX: 'data' leaks */
	msn_cmdproc_send_trans(cmdproc, trans);
}

static void
update_contact_network(MsnSession *session, const char *passport, MsnNetwork network, gpointer unused)

			purple_debug_info("msn", "FQY response says %s is from network %d\n",
			                  passport, network);
			if (cmd->trans->data) {
				MsnFqyCbData *fqy_data = cmd->trans->data;
				fqy_data->cb(session, passport, network, fqy_data->data);
				/* TODO: This leaks, but the server responds to FQY multiple times, so we
				         can't free it yet. We need to figure out somewhere else to do so.
				g_free(fqy_data); */
			}

			g_free(passport);
		}
	}

28353:6a23d7f84143

	data->cb = cb;
	data->data = cb_data;

	trans = msn_transaction_new(cmdproc, "FQY", "%d", payload_len);
	msn_transaction_set_payload(trans, payload, payload_len);
	msn_transaction_set_data(trans, data);
	msn_transaction_set_data_free(trans, g_free);
	msn_cmdproc_send_trans(cmdproc, trans);
}

static void
update_contact_network(MsnSession *session, const char *passport, MsnNetwork network, gpointer unused)

			purple_debug_info("msn", "FQY response says %s is from network %d\n",
			                  passport, network);
			if (cmd->trans->data) {
				MsnFqyCbData *fqy_data = cmd->trans->data;
				fqy_data->cb(session, passport, network, fqy_data->data);
				/* Don't free fqy_data yet since the server responds to FQY multipe times.
				   It will be freed when cmd->trans is freed. */
			}

			g_free(passport);
		}
	}