Mercurial > pidgin
view libpurple/debug.h @ 15976:a6a79b8616bf
I commonly see a crash in which socket_ready_cb(), shortly after a laptop wakes from sleep, is passed invalid (previously freed) connect_data. It looks like this:
Thread 0 Crashed:
0 libobjc.A.dylib 0x90a59380 objc_msgSend 16
1 Libgaim 0x0fe23bcd gaim_proxy_connect_data_disconnect 172
2 Libgaim 0x0fe23d63 socket_ready_cb 199
3 com.apple.CoreFoundation 0x90843ffd __CFSocketDoCallback 551
(objc_msgSend is how ObjC routes messages... it's being called because connect_data->cconnect_cb is invalid).
It appears that when this crash happens, the socket is marked as ready just before the computer sleeps; on the next run loop, the callback will be called [socket_ready_cb()]. The computer sleeps and every account is disconnected first, which calls gaim_proxy_connect_cancel_with_handle(), destroying the connect_data. When it awakens, it calls socket_ready_cb() and the crash occurs.
I've added PURPLE_PROXY_CONNECT_DATA_IS_VALID, which takes advantage of the fact that all valid connect_data objects are stored in the handles GSList, just as PURPLE_GAIM_CONNECTION_IS_VALID works.
author | Evan Schoenberg <evan.s@dreskin.net> |
---|---|
date | Sun, 01 Apr 2007 02:17:06 +0000 |
parents | 32c366eeeb99 |
children | 6dc5dc83a61b |
line wrap: on
line source
/** * @file debug.h Debug API * @ingroup core * * purple * * Purple is the legal property of its developers, whose names are too numerous * to list here. Please refer to the COPYRIGHT file distributed with this * source distribution. * * 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ #ifndef _PURPLE_DEBUG_H_ #define _PURPLE_DEBUG_H_ #include <glib.h> #include <stdarg.h> /** * Debug levels. */ typedef enum { PURPLE_DEBUG_ALL = 0, /**< All debug levels. */ PURPLE_DEBUG_MISC, /**< General chatter. */ PURPLE_DEBUG_INFO, /**< General operation Information. */ PURPLE_DEBUG_WARNING, /**< Warnings. */ PURPLE_DEBUG_ERROR, /**< Errors. */ PURPLE_DEBUG_FATAL /**< Fatal errors. */ } PurpleDebugLevel; /** * Debug UI operations. */ typedef struct { void (*print)(PurpleDebugLevel level, const char *category, const char *arg_s); } PurpleDebugUiOps; #ifdef __cplusplus extern "C" { #endif /**************************************************************************/ /** @name Debug API */ /**************************************************************************/ /** * Outputs debug information. * * @param level The debug level. * @param category The category (or @c NULL). * @param format The format string. */ void purple_debug(PurpleDebugLevel level, const char *category, const char *format, ...); /** * Outputs misc. level debug information. * * This is a wrapper for purple_debug(), and uses PURPLE_DEBUG_MISC as * the level. * * @param category The category (or @c NULL). * @param format The format string. * * @see purple_debug() */ void purple_debug_misc(const char *category, const char *format, ...); /** * Outputs info level debug information. * * This is a wrapper for purple_debug(), and uses PURPLE_DEBUG_INFO as * the level. * * @param category The category (or @c NULL). * @param format The format string. * * @see purple_debug() */ void purple_debug_info(const char *category, const char *format, ...); /** * Outputs warning level debug information. * * This is a wrapper for purple_debug(), and uses PURPLE_DEBUG_WARNING as * the level. * * @param category The category (or @c NULL). * @param format The format string. * * @see purple_debug() */ void purple_debug_warning(const char *category, const char *format, ...); /** * Outputs error level debug information. * * This is a wrapper for purple_debug(), and uses PURPLE_DEBUG_ERROR as * the level. * * @param category The category (or @c NULL). * @param format The format string. * * @see purple_debug() */ void purple_debug_error(const char *category, const char *format, ...); /** * Outputs fatal error level debug information. * * This is a wrapper for purple_debug(), and uses PURPLE_DEBUG_ERROR as * the level. * * @param category The category (or @c NULL). * @param format The format string. * * @see purple_debug() */ void purple_debug_fatal(const char *category, const char *format, ...); /** * Enable or disable printing debug output to the console. * * @param enabled TRUE to enable debug output or FALSE to disable it. */ void purple_debug_set_enabled(gboolean enabled); /** * Check if console debug output is enabled. * * @return TRUE if debuggin is enabled, FALSE if it is not. */ gboolean purple_debug_is_enabled(void); /*@}*/ /**************************************************************************/ /** @name UI Registration Functions */ /**************************************************************************/ /*@{*/ /** * Sets the UI operations structure to be used when outputting debug * information. * * @param ops The UI operations structure. */ void purple_debug_set_ui_ops(PurpleDebugUiOps *ops); /** * Returns the UI operations structure used when outputting debug * information. * * @return The UI operations structure in use. */ PurpleDebugUiOps *purple_debug_get_ui_ops(void); /*@}*/ /**************************************************************************/ /** @name Debug Subsystem */ /**************************************************************************/ /*@{*/ /** * Initializes the debug subsystem. */ void purple_debug_init(void); /*@}*/ #ifdef __cplusplus } #endif #endif /* _PURPLE_DEBUG_H_ */