Remove nugatory EC_TTL_EXPIRED error before idle connection closes.v1.0.3
commit375e5a66d7ea444405c2bc3e4f6b450b3a9bfa39
authorben <bschofield@gmail.com>
Tue, 1 Feb 2022 19:01:31 +0000 (1 19:01 +0000)
committerrofl0r <rofl0r@users.noreply.github.com>
Tue, 1 Feb 2022 19:14:17 +0000 (1 19:14 +0000)
tree1afcaee7a73613188afc49929654337df5345254
parent2702f853e144430743aa9919aedd6c20bdf7ea5a
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 #53.
sockssrv.c