mirror of
https://github.com/SagerNet/sing-box.git
synced 2025-02-16 14:24:31 +00:00
Fix UDP DNS response not truncated
This commit is contained in:
parent
f76b21b02c
commit
17aebc56c1
|
@ -241,7 +241,8 @@ func (d *DNS) newPacketConnection(ctx context.Context, conn N.PacketConn, readWa
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
timeout.Update()
|
timeout.Update()
|
||||||
responseBuffer := buf.NewPacket()
|
response = truncateDNSMessage(response, 512) // TODO: add an option to custom UDP buffer size
|
||||||
|
responseBuffer := buf.NewSize(dns.FixedPacketSize)
|
||||||
responseBuffer.Resize(1024, 0)
|
responseBuffer.Resize(1024, 0)
|
||||||
n, err := response.PackBuffer(responseBuffer.FreeBytes())
|
n, err := response.PackBuffer(responseBuffer.FreeBytes())
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
@ -263,3 +264,21 @@ func (d *DNS) newPacketConnection(ctx context.Context, conn N.PacketConn, readWa
|
||||||
})
|
})
|
||||||
return group.Run(fastClose)
|
return group.Run(fastClose)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func truncateDNSMessage(response *mDNS.Msg, maxLen int) *mDNS.Msg {
|
||||||
|
responseLen := response.Len()
|
||||||
|
if responseLen <= maxLen {
|
||||||
|
return response
|
||||||
|
}
|
||||||
|
response = response.Copy()
|
||||||
|
for len(response.Answer) > 0 && responseLen > maxLen {
|
||||||
|
response.Answer = response.Answer[:len(response.Answer)-1]
|
||||||
|
response.Truncated = true
|
||||||
|
responseLen = response.Len()
|
||||||
|
}
|
||||||
|
if responseLen > maxLen {
|
||||||
|
response.Ns = nil
|
||||||
|
response.Extra = nil
|
||||||
|
}
|
||||||
|
return response
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in a new issue