[BUG]客户端forward模式，转发udp报错：failed to parse udp packet addr

simplerick-simplefun commented 2 years ago

[x] 我确定我已经尝试多次复现此次问题，并且将会提供涉及此问题的系统和网络环境，软件及其版本。

简单描述这个 Bug

使用trojan-go客户端，设置forward模式，转发udp数据包，报错failed to parse udp packet addr | unable to read ATYP

如何复现这个 Bug

1.trojan-go客户端A，配置forward模式，监听本地127.0.0.1:Ax，连接服务端trojan-go地址B:By，转发到C:Cz 2.本地程序D发送udp数据包到127.0.0.1:Ax 3.预期结果：程序D正常C:Cz连接 4.实际结果：trojan-go客户端A报错：failed to parse udp packet addr | unable to read ATYP

注意：如果把客户端改成原版trojan，其他均不变，则一切正常。程序D能够正常与C:Cz连接。

服务器和客户端环境信息

客户端Windows 11

服务端和客户端日志

[ERROR] 2022/04/21 18:40:06 github.com/p4gefau1t/trojan-go/tunnel/router.(*PacketConn).packetLoop.func1:conn.go:37 router packetConn error failed to parse udp packet addr | unable to read ATYP | read tcp 192.168.50.219:54647->67.230.191.69:443: wsarecv: An established connection was aborted by the software in your host machine.

服务端和客户端配置文件

{
    "run_type": "forward",
    "local_addr": "127.0.0.1",
    "local_port": [trojan-go_local_listen_port],
    "remote_addr": [trojan-go_server_addr],
    "remote_port": [trojan-go_server_port],
    "target_addr": [udp_parcket_destination_addr],
    "target_port": [udp_parcket_destination_port],
    "udp_timeout": 600,
    "password": [
        "password"
    ],
    "mux": {
        "enabled": false
    },
    "ssl": {
        "sni": [trojan-go_server_sni]
    }
}

服务端和客户端版本信息

Trojan-Go v0.10.6 Go Version: go1.17.1 OS/Arch: windows/amd64 Git Commit: 2dc60f52e79ff8b910e78e444f1e80678e936450

Developed by PageFault (p4gefau1t) Licensed under GNU General Public License version 3 GitHub Repository: https://github.com/p4gefau1t/trojan-go Trojan-Go Documents: https://p4gefau1t.github.io/trojan-go/

其他信息

如果把客户端改成原版trojan，其他均不变，则一切正常。 forward模式下，客户端应该按照配置文件，把数据包转发到设置的目的地IP:port，而不应该去parse这个来源数据包是去哪里的。因为来源数据包本身的目的地是trojan监听的地址：端口，预期目的地是配置文件中设置要转发的地址：端口。数据包不是通过iptable/tproxy转发到trojan-go客户端的，所以不可能parse出来udp packet addr。我没有测试forward模式下tcp数据包转发，但怀疑存在相同问题。

simplerick-simplefun commented 2 years ago

大概看了一下，应该是这里的问题： https://github.com/p4gefau1t/trojan-go/blob/7f8d638260b0666a288a34ec2112c6da314f8b57/tunnel/trojan/packet.go#L58 这一行就是读取目标地址的，但是forward转发没有地址会跳到这里： https://github.com/p4gefau1t/trojan-go/blob/7f8d638260b0666a288a34ec2112c6da314f8b57/tunnel/metadata.go#L141 但是这个ReadWithMetadata(payload []byte)应该怎么改，加一个if forward的condition之后，length那里怎么处理，我就不知道了。 https://github.com/p4gefau1t/trojan-go/blob/7f8d638260b0666a288a34ec2112c6da314f8b57/tunnel/trojan/packet.go#L54