source: protocols/jabber/conference.c @ bc49ec2

Last change on this file since bc49ec2 was e73a501, checked in by Wilmer van der Gaast <wilmer@…>, at 2010-05-10T23:35:58Z

Fixed a long-standing bug where non-anonymous Jabber chatrooms could cause
full JIDs to be added to the local contact list, but not removed when
leaving the room, eventually triggering errors like the one in #499. People
kept sending me these dumps and I finally took time to make what turned out
to be this one-line fix.

This error should never appear again unless people are doing really crazy
stuff.
  • Property mode set to 100644
File size: 11.1 KB
Line 
1/***************************************************************************\
2*                                                                           *
3*  BitlBee - An IRC to IM gateway                                           *
4*  Jabber module - Conference rooms                                         *
5*                                                                           *
6*  Copyright 2007 Wilmer van der Gaast <wilmer@gaast.net>                   *
7*                                                                           *
8*  This program is free software; you can redistribute it and/or modify     *
9*  it under the terms of the GNU General Public License as published by     *
10*  the Free Software Foundation; either version 2 of the License, or        *
11*  (at your option) any later version.                                      *
12*                                                                           *
13*  This program is distributed in the hope that it will be useful,          *
14*  but WITHOUT ANY WARRANTY; without even the implied warranty of           *
15*  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the            *
16*  GNU General Public License for more details.                             *
17*                                                                           *
18*  You should have received a copy of the GNU General Public License along  *
19*  with this program; if not, write to the Free Software Foundation, Inc.,  *
20*  51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.              *
21*                                                                           *
22\***************************************************************************/
23
24#include "jabber.h"
25
26static xt_status jabber_chat_join_failed( struct im_connection *ic, struct xt_node *node, struct xt_node *orig );
27
28struct groupchat *jabber_chat_join( struct im_connection *ic, const char *room, const char *nick, const char *password )
29{
30        struct jabber_chat *jc;
31        struct xt_node *node;
32        struct groupchat *c;
33        char *roomjid;
34       
35        roomjid = g_strdup_printf( "%s/%s", room, nick );
36        node = xt_new_node( "x", NULL, NULL );
37        xt_add_attr( node, "xmlns", XMLNS_MUC );
38        if( password )
39                xt_add_child( node, xt_new_node( "password", password, NULL ) );
40        node = jabber_make_packet( "presence", NULL, roomjid, node );
41        jabber_cache_add( ic, node, jabber_chat_join_failed );
42       
43        if( !jabber_write_packet( ic, node ) )
44        {
45                g_free( roomjid );
46                return NULL;
47        }
48       
49        jc = g_new0( struct jabber_chat, 1 );
50        jc->name = jabber_normalize( room );
51       
52        if( ( jc->me = jabber_buddy_add( ic, roomjid ) ) == NULL )
53        {
54                g_free( roomjid );
55                g_free( jc->name );
56                g_free( jc );
57                return NULL;
58        }
59       
60        /* roomjid isn't normalized yet, and we need an original version
61           of the nick to send a proper presence update. */
62        jc->my_full_jid = roomjid;
63       
64        c = imcb_chat_new( ic, room );
65        c->data = jc;
66       
67        return c;
68}
69
70static xt_status jabber_chat_join_failed( struct im_connection *ic, struct xt_node *node, struct xt_node *orig )
71{
72        struct jabber_error *err;
73        struct jabber_buddy *bud;
74        char *room;
75       
76        room = xt_find_attr( orig, "to" );
77        bud = jabber_buddy_by_jid( ic, room, 0 );
78        err = jabber_error_parse( xt_find_node( node->children, "error" ), XMLNS_STANZA_ERROR );
79        if( err )
80        {
81                imcb_error( ic, "Error joining groupchat %s: %s%s%s", room, err->code,
82                            err->text ? ": " : "", err->text ? err->text : "" );
83                jabber_error_free( err );
84        }
85        if( bud )
86                jabber_chat_free( jabber_chat_by_jid( ic, bud->bare_jid ) );
87       
88        return XT_HANDLED;
89}
90
91struct groupchat *jabber_chat_by_jid( struct im_connection *ic, const char *name )
92{
93        char *normalized = jabber_normalize( name );
94        struct groupchat *ret;
95        struct jabber_chat *jc;
96       
97        for( ret = ic->groupchats; ret; ret = ret->next )
98        {
99                jc = ret->data;
100                if( strcmp( normalized, jc->name ) == 0 )
101                        break;
102        }
103        g_free( normalized );
104       
105        return ret;
106}
107
108void jabber_chat_free( struct groupchat *c )
109{
110        struct jabber_chat *jc = c->data;
111       
112        jabber_buddy_remove_bare( c->ic, jc->name );
113       
114        g_free( jc->my_full_jid );
115        g_free( jc->name );
116        g_free( jc );
117       
118        imcb_chat_free( c );
119}
120
121int jabber_chat_msg( struct groupchat *c, char *message, int flags )
122{
123        struct im_connection *ic = c->ic;
124        struct jabber_chat *jc = c->data;
125        struct xt_node *node;
126       
127        jc->flags |= JCFLAG_MESSAGE_SENT;
128       
129        node = xt_new_node( "body", message, NULL );
130        node = jabber_make_packet( "message", "groupchat", jc->name, node );
131       
132        if( !jabber_write_packet( ic, node ) )
133        {
134                xt_free_node( node );
135                return 0;
136        }
137        xt_free_node( node );
138       
139        return 1;
140}
141
142int jabber_chat_topic( struct groupchat *c, char *topic )
143{
144        struct im_connection *ic = c->ic;
145        struct jabber_chat *jc = c->data;
146        struct xt_node *node;
147       
148        node = xt_new_node( "subject", topic, NULL );
149        node = jabber_make_packet( "message", "groupchat", jc->name, node );
150       
151        if( !jabber_write_packet( ic, node ) )
152        {
153                xt_free_node( node );
154                return 0;
155        }
156        xt_free_node( node );
157       
158        return 1;
159}
160
161int jabber_chat_leave( struct groupchat *c, const char *reason )
162{
163        struct im_connection *ic = c->ic;
164        struct jabber_chat *jc = c->data;
165        struct xt_node *node;
166       
167        node = xt_new_node( "x", NULL, NULL );
168        xt_add_attr( node, "xmlns", XMLNS_MUC );
169        node = jabber_make_packet( "presence", "unavailable", jc->my_full_jid, node );
170       
171        if( !jabber_write_packet( ic, node ) )
172        {
173                xt_free_node( node );
174                return 0;
175        }
176        xt_free_node( node );
177       
178        return 1;
179}
180
181void jabber_chat_invite( struct groupchat *c, char *who, char *message )
182{
183        struct xt_node *node;
184        struct im_connection *ic = c->ic;
185        struct jabber_chat *jc = c->data;
186
187        node = xt_new_node( "reason", message, NULL ); 
188
189        node = xt_new_node( "invite", NULL, node );
190        xt_add_attr( node, "to", who ); 
191
192        node = xt_new_node( "x", NULL, node ); 
193        xt_add_attr( node, "xmlns", XMLNS_MUC_USER ); 
194       
195        node = jabber_make_packet( "message", NULL, jc->name, node ); 
196
197        jabber_write_packet( ic, node ); 
198
199        xt_free_node( node );
200}
201
202/* Not really the same syntax as the normal pkt_ functions, but this isn't
203   called by the xmltree parser directly and this way I can add some extra
204   parameters so we won't have to repeat too many things done by the caller
205   already. */
206void jabber_chat_pkt_presence( struct im_connection *ic, struct jabber_buddy *bud, struct xt_node *node )
207{
208        struct groupchat *chat;
209        struct xt_node *c;
210        char *type = xt_find_attr( node, "type" );
211        struct jabber_chat *jc;
212        char *s;
213       
214        if( ( chat = jabber_chat_by_jid( ic, bud->bare_jid ) ) == NULL )
215        {
216                /* How could this happen?? We could do kill( self, 11 )
217                   now or just wait for the OS to do it. :-) */
218                return;
219        }
220       
221        jc = chat->data;
222       
223        if( type == NULL && !( bud->flags & JBFLAG_IS_CHATROOM ) )
224        {
225                bud->flags |= JBFLAG_IS_CHATROOM;
226                /* If this one wasn't set yet, this buddy just joined the chat.
227                   Slightly hackish way of finding out eh? ;-) */
228               
229                /* This is pretty messy... Here it sets ext_jid to the real
230                   JID of the participant. Works for non-anonymized channels.
231                   Might break if someone joins a chat twice, though. */
232                for( c = node->children; ( c = xt_find_node( c, "x" ) ); c = c->next )
233                        if( ( s = xt_find_attr( c, "xmlns" ) ) &&
234                            ( strcmp( s, XMLNS_MUC_USER ) == 0 ) )
235                        {
236                                struct xt_node *item;
237                               
238                                item = xt_find_node( c->children, "item" );
239                                if( ( s = xt_find_attr( item, "jid" ) ) )
240                                {
241                                        /* Yay, found what we need. :-) */
242                                        bud->ext_jid = jabber_normalize( s );
243                                        break;
244                                }
245                        }
246               
247                /* Make up some other handle, if necessary. */
248                if( bud->ext_jid == NULL )
249                {
250                        if( bud == jc->me )
251                        {
252                                bud->ext_jid = jabber_normalize( ic->acc->user );
253                        }
254                        else
255                        {
256                                int i;
257                               
258                                /* Don't want the nick to be at the end, so let's
259                                   think of some slightly different notation to use
260                                   for anonymous groupchat participants in BitlBee. */
261                                bud->ext_jid = g_strdup_printf( "%s=%s", bud->resource, bud->bare_jid );
262                               
263                                /* And strip any unwanted characters. */
264                                for( i = 0; bud->resource[i]; i ++ )
265                                        if( bud->ext_jid[i] == '=' || bud->ext_jid[i] == '@' )
266                                                bud->ext_jid[i] = '_';
267                               
268                                /* Some program-specific restrictions. */
269                                imcb_clean_handle( ic, bud->ext_jid );
270                        }
271                        bud->flags |= JBFLAG_IS_ANONYMOUS;
272                }
273               
274                if( bud != jc->me && bud->flags & JBFLAG_IS_ANONYMOUS )
275                {
276                        /* If JIDs are anonymized, add them to the local
277                           list for the duration of this chat. */
278                        imcb_add_buddy( ic, bud->ext_jid, NULL );
279                        imcb_buddy_nick_hint( ic, bud->ext_jid, bud->resource );
280                }
281               
282                s = strchr( bud->ext_jid, '/' );
283                if( s ) *s = 0; /* Should NEVER be NULL, but who knows... */
284                imcb_chat_add_buddy( chat, bud->ext_jid );
285                if( s ) *s = '/';
286        }
287        else if( type ) /* type can only be NULL or "unavailable" in this function */
288        {
289                if( ( bud->flags & JBFLAG_IS_CHATROOM ) && bud->ext_jid )
290                {
291                        s = strchr( bud->ext_jid, '/' );
292                        if( s ) *s = 0;
293                        imcb_chat_remove_buddy( chat, bud->ext_jid, NULL );
294                        if( bud != jc->me && bud->flags & JBFLAG_IS_ANONYMOUS )
295                                imcb_remove_buddy( ic, bud->ext_jid, NULL );
296                        if( s ) *s = '/';
297                }
298               
299                if( bud == jc->me )
300                        jabber_chat_free( chat );
301        }
302}
303
304void jabber_chat_pkt_message( struct im_connection *ic, struct jabber_buddy *bud, struct xt_node *node )
305{
306        struct xt_node *subject = xt_find_node( node->children, "subject" );
307        struct xt_node *body = xt_find_node( node->children, "body" );
308        struct groupchat *chat = bud ? jabber_chat_by_jid( ic, bud->bare_jid ) : NULL;
309        struct jabber_chat *jc = chat ? chat->data : NULL;
310        char *s;
311       
312        if( bud == NULL || ( jc && ~jc->flags & JCFLAG_MESSAGE_SENT && bud == jc->me ) )
313        {
314                char *nick;
315               
316                if( body == NULL || body->text_len == 0 )
317                        /* Meh. Empty messages aren't very interesting, no matter
318                           how much some servers love to send them. */
319                        return;
320               
321                s = xt_find_attr( node, "from" ); /* pkt_message() already NULL-checked this one. */
322                nick = strchr( s, '/' );
323                if( nick )
324                {
325                        /* If this message included a resource/nick we don't know,
326                           we might still know the groupchat itself. */
327                        *nick = 0;
328                        chat = jabber_chat_by_jid( ic, s );
329                        *nick = '/';
330                       
331                        nick ++;
332                }
333                else
334                {
335                        /* message.c uses the EXACT_JID option, so bud should
336                           always be NULL here for bare JIDs. */
337                        chat = jabber_chat_by_jid( ic, s );
338                }
339               
340                if( nick == NULL )
341                {
342                        /* This is fine, the groupchat itself isn't in jd->buddies. */
343                        if( chat )
344                                imcb_chat_log( chat, "From conference server: %s", body->text );
345                        else
346                                imcb_log( ic, "System message from unknown groupchat %s: %s", s, body->text );
347                }
348                else
349                {
350                        /* This can happen too, at least when receiving a backlog when
351                           just joining a channel. */
352                        if( chat )
353                                imcb_chat_log( chat, "Message from unknown participant %s: %s", nick, body->text );
354                        else
355                                imcb_log( ic, "Groupchat message from unknown JID %s: %s", s, body->text );
356                }
357               
358                return;
359        }
360        else if( chat == NULL )
361        {
362                /* How could this happen?? We could do kill( self, 11 )
363                   now or just wait for the OS to do it. :-) */
364                return;
365        }
366       
367        if( subject )
368        {
369                s = strchr( bud->ext_jid, '/' );
370                if( s ) *s = 0;
371                imcb_chat_topic( chat, bud->ext_jid, subject->text_len > 0 ?
372                                 subject->text : NULL, jabber_get_timestamp( node ) );
373                if( s ) *s = '/';
374        }
375        if( body && body->text_len > 0 )
376        {
377                s = strchr( bud->ext_jid, '/' );
378                if( s ) *s = 0;
379                imcb_chat_msg( chat, bud->ext_jid, body->text, 0, jabber_get_timestamp( node ) );
380                if( s ) *s = '/';
381        }
382}
Note: See TracBrowser for help on using the repository browser.