<div class="gmail_quote">On Thu, Nov 26, 2009 at 1:30 PM, Paul Aurich <span dir="ltr"><<a href="mailto:paul@darkrain42.org">paul@darkrain42.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Nov 25, 2009, at 11:08, <a href="mailto:nosnilmot@pidgin.im">nosnilmot@pidgin.im</a> wrote:<br>
> Modified files:<br>
>        libpurple/protocols/jabber/jabber.c<br>
><br>
> ChangeLog:<br>
><br>
> Add some safety to jabber_send_signal_cb in case someone else was listening<br>
> to jabber-sending-xmlnode and caused the connection to get disconnected<br>
> before the signal got to calling jabber_send_signal_cb. Should fix<br>
> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=540835" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=540835</a><br>
<br>
<br>
I'm not convinced this is how the crash is occurring.<br>
<br>
>From the backtrace, frame 10 indicates this code path is being triggered by a FT request denial.  I'm guessing the FT request dialog isn't closed when an account disconnects (I don't have Pidgin handy, so I can't confirm that, but the request dialog uses the xfer as the purple_request_ handle, not gc).<br>
</blockquote><div><br>While I can't say it's the cause of this crash in particular, I can say I have noticed this behaviour. Whether it's because of NM disconnecting or just the account itself, file transfer requests tend to stick around in the FT dialog until you do something about them (whether it's for sending or receiving).<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
It seems like a better (and more general) solution would be to close all xfers when an account disconnects, otherwise crashes like this probably exist in other prpls (unless they also use PURPLE_CONNECTION_IS_VALID).<br>

<br>
~Paul<br></blockquote></div>