Tags: huxs001/microsocks
Tags
Remove nugatory EC_TTL_EXPIRED error before idle connection closes. In the case where connect_socks_target() succeeds, the proxy client receives an EC_SUCCESS message and the client thread passes control to copyloop(), which then proxies application-layer data. If both the client and remote connections are idle for too long, the call to poll() inside copyloop() returns zero, giving microsocks the opportunity to reap the inactive connection. Before closing the idle connection an EC_TTL_EXPIRED message is returned to the client. Since the client has already previously received EC_SUCCESS, it believes it is receiving application-layer data from the proxied target. This causes the user to receive a false \x05\x06\x00\x01\x00\x00\x00\x00\x00\x00 data sequence before the connection is closed. This commit removes the call to send_error() that returns the EC_TTL_EXPIRED message for an idle connection. An EC_TTL_EXPIRED error can still correctly be returned in the case where the initial socket() call to the remote address returns ETIMEDOUT. Fixes issue rofl0r#53.
fix rfc1929 user/pass auth subnegotation response version field the version field is repurposed here to mean "version of the subnegotation", which in this case is 1, not 5, as intuitively expected. interestingly, most socks5 clients don't check the version respond, rather only the success field.