comparison libpurple/protocols/myspace/myspace.c @ 17664:8ee3fd09a543

Remove lots of debugging statements, no longer needed. Along with 172d2f1fc811dba4cf65d127b27a799d1e121b58, this patch (I believe) fixes #193. The problem was that msim_close() was being called twice. Removed spurious calls, and no longer appears to crash. This now works: * Login with an invalid password. * `Protocol error, code 260: The password provided is incorrect.` (good) * Disable, and re-enable account. * Receive the same error message (good, no crash). * Repeat previous two steps ''ad nauseum'', with no crash. Closes #193.

17663:814992cefc9c

 * @param session The session to destroy.
 */
void 
msim_session_destroy(MsimSession *session)
{
	purple_debug_info("msim", ">>> going to check if valid\n");

    g_return_if_fail(MSIM_SESSION_VALID(session));
	
	purple_debug_info("msim", ">>> going to set magic\n");

    session->magic = -1;

	purple_debug_info("msim", ">>> going to free rxbuf\n");
    g_free(session->rxbuf);

	/* TODO: Remove. */
	purple_debug_info("msim", ">>> going to free user_lookup_cb\n");
	g_hash_table_destroy(session->user_lookup_cb);
	purple_debug_info("msim", ">>> going to free user_lookup_cb_data\n");
	g_hash_table_destroy(session->user_lookup_cb_data);
	
	purple_debug_info("msim", ">>> going to free session itself\n");
    g_free(session);
}
                 
/** 
 * Close the connection.

void 
msim_close(PurpleConnection *gc)
{
	MsimSession *session;

	purple_debug_info("msim", "msim_close: closing, gc=0x%x\n", gc);

	if (gc == NULL)
		return;

	purple_debug_info("msim", "msim_close: dereferencing gc->proto_data\n");
	session = (MsimSession *)gc->proto_data;
	if (session == NULL)
		return;

	gc->proto_data = NULL;

	purple_debug_info("msim", "msim_close: checking if session is valid\n");
	if (!MSIM_SESSION_VALID(session))
		return;

	purple_debug_info("msim", "msim_close: about to remove inpa\n");
    if (session->gc->inpa)
		purple_input_remove(session->gc->inpa);

	purple_debug_info("msim", "msim_close: ready to destroy session\n");
    msim_session_destroy(session);
}


/**

17664:8ee3fd09a543

 * @param session The session to destroy.
 */
void 
msim_session_destroy(MsimSession *session)
{
    g_return_if_fail(MSIM_SESSION_VALID(session));
	
    session->magic = -1;

    g_free(session->rxbuf);

	/* TODO: Remove. */
	g_hash_table_destroy(session->user_lookup_cb);
	g_hash_table_destroy(session->user_lookup_cb_data);
	
    g_free(session);
}
                 
/** 
 * Close the connection.

void 
msim_close(PurpleConnection *gc)
{
	MsimSession *session;

	if (gc == NULL)
		return;

	session = (MsimSession *)gc->proto_data;
	if (session == NULL)
		return;

	gc->proto_data = NULL;

	if (!MSIM_SESSION_VALID(session))
		return;

    if (session->gc->inpa)
		purple_input_remove(session->gc->inpa);

    msim_session_destroy(session);
}


/**