Changeset 995913b for protocols/jabber/iq.c

Timestamp:

2006-10-02T17:46:57Z (18 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

6266fca

Parents:

501b4e0

Message:

Added some error handling for the (not very complete yet) privacy list code.

File:

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

protocols/jabber/iq.c

@@ -148 +148 @@
         {
                 struct xt_node *c;
+                
                 if( !( jd->flags & JFLAG_AUTHENTICATED ) &&
                     ( c = xt_find_node( orig->children, "query" ) ) &&
@@ -199 +200 @@
         {
                 if( !( jd->flags & JFLAG_AUTHENTICATED ) &&
+                      orig &&
                     ( c = xt_find_node( orig->children, "query" ) ) &&
                     ( c = xt_find_node( c->children, "username" ) ) &&
@@ -207 +209 @@
                         return XT_ABORT;
                 }
-                else if( orig &&
-                         ( c = xt_find_node( orig->children, "query" ) ) &&
-                         ( c = xt_find_node( c->children, "active" ) ) )
-                {
-                        serv_got_crap( gc, "Error while activating privacy list, maybe it doesn't exist" );
+                else if( ( xmlns && strcmp( xmlns, "jabber:iq:privacy" ) == 0 ) ||
+                         ( orig &&
+                           ( c = xt_find_node( orig->children, "query" ) ) &&
+                           ( s = xt_find_attr( c, "xmlns" ) ) &&
+                           strcmp( s, "jabber:iq:privacy" ) == 0 ) )
+                {
+                        /* All errors related to privacy lists. */
+                        if( ( c = xt_find_node( node->children, "error" ) ) == NULL )
+                        {
+                                hide_login_progress_error( gc, "Received malformed error packet" );
+                                signoff( gc );
+                                return XT_ABORT;
+                        }
+                        
+                        if( xt_find_node( c->children, "item-not-found" ) )
+                        {
+                                serv_got_crap( gc, "Error while activating privacy list, maybe it doesn't exist" );
+                                /* Should I do anything else here? */
+                        }
+                        else if( xt_find_node( c->children, "feature-not-implemented" ) )
+                        {
+                                jd->flags |= JFLAG_PRIVACY_BROKEN;
+                                /* Probably there's no need to inform the user.
+                                   We can do that if the user ever tries to use
+                                   the block/allow commands. */
+                        }
                 }
         }
@@ -327 +350 @@
         node = jabber_make_packet( "iq", "get", NULL, node );
         
-        st = jabber_write_packet( gc, node );
-        
-        xt_free_node( node );
+        jabber_cache_packet( gc, node );
+        st = jabber_write_packet( gc, node );
+        
         return st;
 }
@@ -347 +370 @@
         return jabber_write_packet( gc, node );
 }
+
+char *set_eval_privacy_list( set_t *set, char *value )
+{
+        account_t *acc = set->data;
+        struct jabber_data *jd = acc->gc->proto_data;
+        
+        if( jd->flags & JFLAG_PRIVACY_BROKEN )
+        {
+                serv_got_crap( acc->gc, "Privacy lists not supported by this server" );
+                return NULL;
+        }
+        
+        /* If we're on-line, return NULL and let the server decide if the
+           chosen list is valid. If we're off-line, just accept it and we'll
+           see later (when we connect). */
+        if( acc->gc )
+                jabber_set_privacy( acc->gc, value );
+        
+        return acc->gc ? NULL : value;
+}