[Pidgin] #9271: jabber_parse_close_stream causes XML parse error with prosody
Pidgin
trac at pidgin.im
Thu Jun 25 20:24:04 EDT 2009
#9271: jabber_parse_close_stream causes XML parse error with prosody
------------------------+---------------------------------------------------
Reporter: darkrain42 | Owner: deryni
Type: defect | Status: closed
Milestone: 2.6.0 | Component: XMPP
Version: 2.5.6 | Resolution: fixed
Keywords: |
------------------------+---------------------------------------------------
Comment(by rekkanoryo at pidgin.im):
(In [6aecd5e7eb4b4ced28c1c4c5e50386030abfe1bd]):[[BR]]
In addition to the plucks listed below, I also changelogged these items.
*** Plucked rev 7c614b09b72037ebd86c03fbbeac7794a155547b
(darkrain42 at pidgin.im):
disapproval of revision 'd34a15898cacf017df9190ca02741880be8dd940'
xmlParseChunk is not reentrant. This /just happened/ to work for
'<proceed/>' but not '<proceed></proceed>'. Closes #9271.
*** Plucked rev 8806e5fba5cdb38ef9be6799e7e829dc2a762a69
(markdoliner at pidgin.im):
Stop exiting early from this function. I don't believe it harms anything
to continue executing even if body is null, and if we exit early then
we don't call purple_connection_set_state(session->gc, PURPLE_CONNECTED)
and the account never gets set to \"connected.\" This seems to affect
any MySpace account that has an empty buddy list.
I'm pretty sure this used to work fine, so I'm assuming MySpace tweaked
the data sent to the client when the user has an empty buddy list.
*** Plucked rev 08979461f8fd9f122c665f3ee192c20788dd1bca
(markdoliner at pidgin.im):
Change the way we parse messages on MySpace a little bit. This
fixes #8846: people using web myspaceIM can't respond to pidgin myspaceIM
For some reason IMs send using the myspace web site are sent so that
they won't become offline messages if the other person is offline.
I'm not really sure why that decision was made.
So now we treat messages with bm 1 the same as messages with bm 121. This
means
we have to combine the function that parses out typing notification with
the function that parses IMs. And we check for typing notifications by
looking for %typing%. Which means if someone sends the IM \"%typing%\"
with
no markup then we'll interpret it as a typing notification. And there's
nothing we can do to differentiate between the two. I asked.
--
Ticket URL: <http://developer.pidgin.im/ticket/9271#comment:3>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list