Changeset 788a1af for protocols/jabber/presence.c

Timestamp:

2006-10-15T20:24:01Z (18 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

d74c644

Parents:

e617b35

Message:

Proper cleanup of jabber buddy structures when removing a buddy from the
list, proper checking (and handling) of events related to buddies that
aren't "hashed" yet, limit checks on priorityto setting, renamed JEP85
to XEP85, support for more XEP85 states.

File:

• protocols/jabber/presence.c (modified) (4 diffs)

protocols/jabber/presence.c

@@ -31 +31 @@
         struct xt_node *c;
         struct jabber_buddy *bud;
+        char *s;
         
         if( !from )
@@ -37 +38 @@
         if( type == NULL )
         {
-                if( strchr( from, '/' ) == NULL )
+                if( ( s = strchr( from, '/' ) ) == NULL )
                 {
                         char *s = xt_to_string( node );
-                        serv_got_crap( gc, "WARNING: Ignoring presence tag with bare JID: %s\n", s );
+                        serv_got_crap( gc, "WARNING: Ignoring presence tag with bare JID: %s", s );
                         g_free( s );
                         return XT_HANDLED;
@@ -47 +48 @@
                 if( !( bud = jabber_buddy_by_jid( gc, from ) ) )
                 {
+                        /* FOR NOW, s still contains the location of the /.
+                           Keep this in mind when changing things here. :-) */
+                        
+                        /* We check if the buddy is in the contact list,
+                           because Jabber servers seem to like to send
+                           presence information of buddies we removed
+                           from our list sometimes, for example... */
+                        
+                        *s = 0;
+                        if( find_buddy( gc, from ) == NULL )
+                        {
+                                *s = '/';
+                                serv_got_crap( gc, "WARNING: Ignoring presence information from unknown JID: %s", from );
+                                return XT_HANDLED;
+                        }
+                        *s = '/';
+                        
                         bud = jabber_buddy_add( gc, from );
                 }
@@ -83 +101 @@
                         serv_got_crap( gc, "WARNING: Ignoring presence tag with bare JID: %s\n", s );
                         g_free( s );
+                        return XT_HANDLED;
+                }
+                
+                if( jabber_buddy_by_jid( gc, from ) == NULL )
+                {
+                        serv_got_crap( gc, "WARNING: Received presence information from unknown JID: %s", from );
                         return XT_HANDLED;
                 }