history-museum/clash
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix: should always drop packet when handle UDP packet (#2659)

Browse Source
This commit is contained in:
yaling888
2023-04-05 14:05:23 +08:00
committed by GitHub
parent 4cd4912749
commit 95bbfe3945
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tunnel/tunnel.go
View File

@ -176,6 +176,7 @@ func resolveMetadata(ctx C.PlainContext, metadata *C.Metadata) (proxy C.Proxy, r
func handleUDPConn(packet *inbound.PacketAdapter) {
metadata := packet.Metadata()
if !metadata.Valid() {
packet.Drop()
log.Warnln("[Metadata] not valid: %#v", metadata)
return
}
@ -188,6 +189,7 @@ func handleUDPConn(packet *inbound.PacketAdapter) {
}
if err := preHandleMetadata(metadata); err != nil {
packet.Drop()
log.Debugln("[Metadata PreHandle] error: %s", err)
return
}
@ -196,8 +198,10 @@ func handleUDPConn(packet *inbound.PacketAdapter) {
if !metadata.Resolved() {
ips, err := resolver.LookupIP(context.Background(), metadata.Host)
if err != nil {
packet.Drop()
return
} else if len(ips) == 0 {
packet.Drop()
return
}
metadata.DstIP = ips[0]
@ -215,6 +219,7 @@ func handleUDPConn(packet *inbound.PacketAdapter) {
}
if handle() {
packet.Drop()
return
}
@ -222,6 +227,8 @@ func handleUDPConn(packet *inbound.PacketAdapter) {
cond, loaded := natTable.GetOrCreateLock(lockKey)
go func() {
defer packet.Drop()
if loaded {
cond.L.Lock()
cond.Wait()