Ticket #12431 (closed patch: fixed)

Opened 19 months ago

Last modified 6 months ago

Patch for MSN crash caused by msn_switchboard_request

Reported by: hanzz Owned by: QuLogic
Milestone: 2.10.0 Component: MSN
Version: 2.7.2 Keywords:
Cc:

Description

Attached crash and log describes the problem:

MsnSwitchBoard *
msn_session_get_swboard(MsnSession *session, const char *username,
						MsnSBFlag flag)
{
....
		swboard = msn_switchboard_new(session);
		swboard->im_user = g_strdup(username);
		msn_switchboard_request_add_user(swboard, username);
....
	return swboard;
}

1. New switchboard is created

2. msn_switchboard_request(swboard); is called, but during this call there is problem with connection to MSN network and account is disconnected (switchboard is freed and no longer exists after that function)

3. non-existing swboard is used => crash

This is more global problem in prpl-msn implementation and my patch fixes just one occurrence, but I would say we can start with my change.

Attachments

crash.txt (7.5 kB) - added by hanzz 19 months ago.
log.txt (1.3 kB) - added by hanzz 19 months ago.
pidgin-2.7.2-msn-request-switchboard.patch (4.0 kB) - added by hanzz 19 months ago.
pidgin-msn-switchboard-2.9.0.patch (4.8 kB) - added by shlomif 6 months ago.
Here is the patch forwarded for pidgin-monotone-HEAD.

Change History

Changed 19 months ago by hanzz

Changed 19 months ago by hanzz

Changed 19 months ago by hanzz

Changed 19 months ago by hanzz

  • owner changed from rekkanoryo to QuLogic
  • component changed from unclassified to MSN

Changed 19 months ago by hanzz

  • milestone set to Patches Needing Review

Changed 17 months ago by salinasv

It looks Ok to me.

Changed 14 months ago by rekkanoryo

  • milestone changed from Patches Needing Review to 2.7.10

With the new MSN code merged, is this patch still relevant? If so, it needs to be shipped in 2.7.10 or rejected.

Changed 14 months ago by rekkanoryo

The cmdproc.[ch] stuff in this patch rejects. Please submit an updated patch (or con Masca or QuLogic into fixing it and committing it before 2.7.10).

Changed 12 months ago by rekkanoryo

  • milestone changed from 2.7.12 to 2.8.0

Changed 6 months ago by shlomif

Here is the patch forwarded for pidgin-monotone-HEAD.

Changed 6 months ago by shlomif

Hi, I forward ported the patch to the monotone HEAD (pidgin-2.9.0-and-above). Please apply it.

Changed 6 months ago by masca@…

  • status changed from new to closed
  • resolution set to fixed

(In [e4038d4f52e6856643df0bce1488a565385a851f]):
Fix a crash when the account is disconnected and we have requested a SB. (Hanzz, ported by shlomif)(Fixes #12431)

Note: See TracTickets for help on using tickets.
All information, including names and email addresses, entered onto this website or sent to mailing lists affiliated with this website will be public. Do not post confidential information, especially passwords!