Menu
▾
▴
[Gaim-patches] [ gaim-Patches-1191553 ] [MSN] msn_p2p_msg() crash fix
|
From: SourceForge.net <no...@so...> - 2005-04-28 13:31:00
|
Patches item #1191553, was opened at 2005-04-28 03:28 Message generated for change (Comment added) made by nosnilmot You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1191553&group_id=235 Category: None Group: None Status: Open Resolution: None Priority: 5 Submitted By: Evan Schoenberg (evands) Assigned to: Stu Tomlinson (nosnilmot) Summary: [MSN] msn_p2p_msg() crash fix Initial Comment: This patch adds g_return_if_fail() safety for the arguments to msn_p2p_msg(), bringing it in line with other msn functions and hopefully fixing a crash in the process. It also corrects a comment typo, "its" rather than "it's". I see a lot of crashes in our crash reporter from MSN which look like: 0 Libgaim 0x0497b374 msn_p2p_msg 0x3c 1 Libgaim 0x049771f8 msg_cmd_post 0x64 2 Libgaim 0x049972a8 read_cb 0x1cc and usually have descriptions of having been idling and come back to find a crash. I haven't been able to reproduce this crash with the patch, though I couldn't reproduce it reliably before, either. It applies cleanly against oldstatus and HEAD. ---------------------------------------------------------------------- >Comment By: Stu Tomlinson (nosnilmot) Date: 2005-04-28 08:30 Message: Logged In: YES user_id=309779 I'm hesitant to accept this without any confirmation that it does anything useful at all. There are a number of similar bugs reported, but they all indicate that slplink->swboard is either NULL or has been free'd at slp.c:738 - and I don't think this patch would affect those situations. Additionally, using g_return_if_fail like this is quite possibly covering up some deeper bug which could just end up causing crashes or other problems elsewhere. In your testing with the patch did you notice whether any of the g_return_if_fails were actually being hit? Do you have *any* pointers on how to reproduce the crash? Can you get your crash reporter to include line numbers? :) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300235&aid=1191553&group_id=235 |