Changeset d97f51b for protocols/msn/soap.c

Timestamp:

2010-08-14T23:23:23Z (14 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

ff27648

Parents:

d912fe4

Message:

Fix issues with logging in with huge SSO tickets (hilariously, the 1024-
byte buffer was just one byte too short, resulting in a \r-terminated
login line and the server waiting for the \n). Also using xt_find_path().

File:

• protocols/msn/soap.c (modified) (5 diffs)

protocols/msn/soap.c

@@ -203 +203 @@
         id += strlen( id ) - 1;
         if( *id == '1' &&
-            ( p = node->parent ) && ( p = p->parent ) &&
-            ( p = xt_find_node( p->children, "wst:RequestedProofToken" ) ) &&
-            ( p = xt_find_node( p->children, "wst:BinarySecret" ) ) &&
+            ( p = xt_find_path( node, "../../wst:RequestedProofToken/wst:BinarySecret" ) ) &&
             p->text )
                 sd->secret = g_strdup( p->text );
@@ -477 +475 @@
         struct im_connection *ic = soap_req->ic;
         
-        if( ( p = node->parent ) && ( p = p->parent ) &&
-            ( p = xt_find_node( p->children, "MemberRole" ) ) )
+        if( ( p = xt_find_path( node, "../../MemberRole" ) ) )
                 role = p->text;
         
@@ -636 +633 @@
         struct msn_soap_req_data *soap_req = data;
         
-        if( ( p = node->parent ) &&
-            ( p = xt_find_node( p->children, "groupId" ) ) )
+        if( ( p = xt_find_path( node, "../groupId" ) ) )
                 id = p->text;
         
@@ -657 +653 @@
         struct im_connection *ic = soap_req->ic;
         
-        if( ( p = node->parent ) &&
-            ( p = xt_find_node( p->children, "contactId" ) ) )
+        if( ( p = xt_find_path( node, "../contactId" ) ) )
                 id = p->text;
         if( ( p = xt_find_node( node->children, "contactType" ) ) )
@@ -676 +671 @@
         }
         
+        if( handle == NULL )
+                return XT_HANDLED;
+        
         if( !( bu = bee_user_by_handle( ic->bee, ic, handle ) ) &&
             !( bu = bee_user_new( ic->bee, ic, handle, 0 ) ) )