Changeset 6fb6410 for protocols

Timestamp:

2006-05-26T15:02:41Z (18 years ago)

Author:

Jelmer Vernooij <jelmer@…>

Branches:

master

Children:

84eddee

Parents:

1cda4f3 (diff), 6048744 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

[merge] Wilmer

Location:

protocols

Files:

• http_client.c (modified) (1 diff)
• msn/sb.c (modified) (2 diffs)
• msn/tables.c (modified) (1 diff)
• oscar/chat.c (modified) (2 diffs)
• oscar/im.c (modified) (1 diff)
• oscar/oscar.c (modified) (2 diffs)

protocols/http_client.c

@@ -240 +240 @@
 
 got_reply:
+        /* Maybe if the webserver is overloaded, or when there's bad SSL
+           support... */
+        if( req->bytes_read == 0 )
+                goto cleanup;
+        
         /* Zero termination is very convenient. */
         req->reply_headers[req->bytes_read] = 0;

protocols/msn/sb.c

@@ -523 +523 @@
                 {
                         msn_sb_destroy( sb );
-                        return( 0 );
-                }
-                if( err->flags & STATUS_FATAL )
+                        return 0;
+                }
+                else if( err->flags & STATUS_FATAL )
                 {
                         signoff( gc );
-                        return( 0 );
-                }
-                if( err->flags & STATUS_SB_IM_SPARE )
+                        return 0;
+                }
+                else if( err->flags & STATUS_SB_IM_SPARE )
                 {
                         if( sb->who )
@@ -553 +553 @@
                                 sb->msgq = NULL;
                         }
+                        
+                        /* Do NOT return 0 here, we want to keep this sb. */
                 }
         }

protocols/msn/tables.c

@@ -127 +127 @@
         
         { 910, "Server is busy",                                        STATUS_FATAL },
-        { 911, "Authentication failed",                                 STATUS_FATAL },
+        { 911, "Authentication failed",                                 STATUS_SB_FATAL | STATUS_FATAL },
         { 912, "Server is busy",                                        STATUS_FATAL },
         { 913, "Not allowed when hiding",                               0 },

protocols/oscar/chat.c

@@ -163 +163 @@
                  the right charset flags, as we also do with normal
                  messages. Hope this will work. :-) */
+        /*
         if (flags & AIM_CHATFLAGS_UNICODE)
                 aimbs_put16(&fr->data, 0x0002);
@@ -171 +172 @@
         
         aimbs_put16(&fr->data, 0x0000);
+        */
         
         /*

protocols/oscar/im.c

@@ -1469 +1469 @@
             case 0x9c:  /* ICQ 5 seems to send this */
                 aim_send_im_ch2_statusmessage(sess, userinfo->sn, args->cookie,
-                        gc->away, sess->aim_icq_state, dc);
+                        gc->away ? gc->away : "", sess->aim_icq_state, dc);
                 break;
 

protocols/oscar/oscar.c

@@ -138 +138 @@
         int i, j;
         char *x = strchr(name, '-');
-        if (!x) return NULL;
+        if (!x) return g_strdup(name);
         x = strchr(++x, '-');
-        if (!x) return NULL;
+        if (!x) return g_strdup(name);
         tmp = g_strdup(++x);
 
@@ -2607 +2607 @@
         int ret;
         static int chat_id = 0;
-        char * chatname = g_new0(char, strlen(gc->username)+4);
-        
-        g_snprintf(chatname, strlen(gc->username) + 4, "%s%d", gc->username, chat_id++);
+        char * chatname;
+        
+        chatname = g_strdup_printf("%s%d", gc->username, chat_id++);
   
         ret = oscar_chat_join(gc, chatname);