[patch] fix IM connections to the same host as the one running bitlbee

[vjt@…]

When you try to connect to a jabberd daemon located on the same host that's running bitlbee, it hangs indefinitely because the gaim_io_connected function is never called: it's called only by the async event infrastructure, but in this case there is no asynchronous event to be triggered as the connection is successfully established before the connect(2) syscall returns.

This small patch fixes this bug:

--- proxy.c.orig        Thu Dec 27 20:07:52 2007
+++ proxy.c     Thu Dec 27 20:01:53 2007
@@ -172,6 +172,9 @@
                        return -1;
                }
        }
+  else {
+    gaim_io_connected(phb, fd, GAIM_INPUT_WRITE);
+  }
 
        return fd;
 }

... and now i'm able to connect to my own jabberd :).

All of this happens on my OpenBSD/i386 server, don't know if other OSs exhibit different behavior.

[vjt@…]

... sorry for the indenting! I use sw=2 and expandtab ..

[wilmer]

Hah, nice catch! This was a known bug, actually: #233. Just that apparently it's closed now, I'm afraid some history got lost there thanks to spamming @&%*)@& lowlifes.

Anyway, now we can really close that bug then. I'll add a milestone to make sure this gets applied in time. Thanks!

[vjt@…]

:) glad to help!

[wilmer]

BTW, I'm not sure if this is always the right solution. There might be code that assumes that the code after proxy_connect() will run before the _connected() event handler, and this patch will break that assumption...

Probably the code in the sockerr_again() block should also be called if the connect() was successful already. I guess I got rid of that part when I made this stuff non-blocking.

[wilmer]

Fixed in r290.