Changeset 68286eb for protocols/jabber/iq.c

Timestamp:

2011-12-20T16:45:53Z (12 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

e14b47b8

Parents:

f9789d4

Message:

Detect JID changes at login time and warn the user about them.

File:

• protocols/jabber/iq.c (modified) (3 diffs)

protocols/jabber/iq.c

@@ -31 +31 @@
 {
         struct im_connection *ic = data;
+        struct jabber_data *jd = ic->proto_data;
         struct xt_node *c, *reply = NULL;
         char *type, *s;
@@ -170 +171 @@
                    was added to (or removed from) the buddy list. AFAIK they're
                    sent even if we added this buddy in our own session. */
-                        int bare_len = strlen( ic->acc->user );
+                        int bare_len = strlen( jd->me );
                         
                         if( ( s = xt_find_attr( node, "from" ) ) == NULL ||
-                            ( strncmp( s, ic->acc->user, bare_len ) == 0 &&
+                            ( strncmp( s, jd->me, bare_len ) == 0 &&
                               ( s[bare_len] == 0 || s[bare_len] == '/' ) ) )
                         {
@@ -343 +344 @@
         {
                 c = xt_find_node( c->children, "jid" );
-                if( c && c->text_len && ( s = strchr( c->text, '/' ) ) &&
-                    strcmp( s + 1, set_getstr( &ic->acc->set, "resource" ) ) != 0 )
+                if( !c || !c->text )
+                {
+                        /* Server is crap, but this is no disaster. */
+                }
+                else if( strncmp( jd->me, c->text, strlen( jd->me ) ) != 0 )
+                {
+                        s = strchr( c->text, '/' );
+                        if( s )
+                                *s = '\0';
+                        jabber_set_me( ic, c->text );
+                        imcb_log( ic, "Server claims your JID is `%s' instead of `%s'. "
+                                  "This mismatch may cause problems with groupchats "
+                                  "and possibly other things.",
+                                  c->text, ic->acc->user );
+                        if( s )
+                                *s = '/';
+                }
+                else if( c && c->text_len && ( s = strchr( c->text, '/' ) ) &&
+                         strcmp( s + 1, set_getstr( &ic->acc->set, "resource" ) ) != 0 )
                         imcb_log( ic, "Server changed session resource string to `%s'", s + 1 );
         }