Changeset 3ab1d31

Timestamp:

2010-04-11T18:12:25Z (15 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

a87e6ba, b308cf9

Parents:

ba7d16f

Message:

Fixing a bug in s5bytestream code to deal with incomplete SOCKS5 messages
and my rewrite of the bs_peek function.

File:

• protocols/jabber/s5bytestream.c (modified) (1 diff)

protocols/jabber/s5bytestream.c

@@ -482 +482 @@
                         else if( ret < sizeof( struct socks5_message ) )
                         {
-                                /* Either a buggy proxy or just one that doesnt regard the SHOULD in XEP-0065
-                                 * saying the reply SHOULD contain the address */
-
-                                ASSERTSOCKOP( ret = recv( fd, &socks5_reply, ret, 0 ), "Dequeuing after MSG_PEEK" );
+                                /* Either a buggy proxy or just one that doesnt regard
+                                 * the SHOULD in XEP-0065 saying the reply SHOULD
+                                 * contain the address. We'll take it, so make sure the
+                                 * next jabber_bs_peek starts with an empty buffer. */
+                                bt->peek_buf_len = 0;
                         }