Tartarus CVS: putty: jacob

tartarus-commits@lists.tartarus.org tartarus-commits@lists.tartarus.org
Fri, 10 Oct 2003 22:20:03 +0100


CVS Root:       /usr/data/cvs
Module:         putty
Changes by:     jacob
Date:           Fri Oct 10 2003  22:20:03 BST

Log message:
In SOCKS5 dynamic forwarding, we were echoing back DST.{ADDR,PORT} as
BND.{ADDR,PORT}. Besides being clearly wrong, correspondence with
Sascha Schwarz suggests that this can confuse some SOCKS5 clients
(Aventail and sockscap32) which seem to assume that the reply must
have ATYP=1 (IPv4 literal) and so get the length wrong.
Now all replies have ATYP=1 with BND.{ADDR,PORT} = 0.0.0.0:0 -- this
apparently follows practice in OpenSSH. (We don't have enough info to
fill these fields in correctly.)

Modified files:
    .               : portfwd.c

Links:
http://cvs.tartarus.org//putty/portfwd.c.diff?r1=1.24&r2=1.25