Explicitly enforce hysteria2 max packet size

nekohasekai · nekohasekai · commit fe8564f7e0b2 · 2024-06-24T10:54:56.000+08:00
diff --git a/hysteria2/packet.go b/hysteria2/packet.go
@@ -177,8 +177,8 @@ func (c *udpPacketConn) WritePacket(buffer *buf.Buffer, destination M.Socksaddr)
 		return net.ErrClosed
 	default:
 	}
-	if buffer.Len() > 0xffff {
-		return &quic.DatagramTooLargeError{MaxDatagramPayloadSize: 0xffff}
+	if buffer.Len() > protocol.MaxUDPSize {
+		return &quic.DatagramTooLargeError{MaxDatagramPayloadSize: protocol.MaxUDPSize}
 	}
 	packetId := uint16(c.packetId.Add(1) % math.MaxUint16)
 	message := allocMessage()
@@ -212,8 +212,8 @@ func (c *udpPacketConn) WriteTo(p []byte, addr net.Addr) (n int, err error) {
 		return 0, net.ErrClosed
 	default:
 	}
-	if len(p) > 0xffff {
-		return 0, &quic.DatagramTooLargeError{MaxDatagramPayloadSize: 0xffff}
+	if len(p) > protocol.MaxUDPSize {
+		return 0, &quic.DatagramTooLargeError{MaxDatagramPayloadSize: protocol.MaxUDPSize}
 	}
 	packetId := uint16(c.packetId.Add(1) % math.MaxUint16)
 	message := allocMessage()

Original file line number	Diff line number	Diff line change
`@@ -177,8 +177,8 @@ func (c udpPacketConn) WritePacket(buffer buf.Buffer, destination M.Socksaddr)`
`177`	`177`	`return net.ErrClosed`
`178`	`178`	`default:`
`179`	`179`	`}`
`180`		`- if buffer.Len() > 0xffff {`
`181`		`- return &quic.DatagramTooLargeError{MaxDatagramPayloadSize: 0xffff}`
	`180`	`+ if buffer.Len() > protocol.MaxUDPSize {`
	`181`	`+ return &quic.DatagramTooLargeError{MaxDatagramPayloadSize: protocol.MaxUDPSize}`
`182`	`182`	`}`
`183`	`183`	`packetId := uint16(c.packetId.Add(1) % math.MaxUint16)`
`184`	`184`	`message := allocMessage()`
`@@ -212,8 +212,8 @@ func (c *udpPacketConn) WriteTo(p []byte, addr net.Addr) (n int, err error) {`
`212`	`212`	`return 0, net.ErrClosed`
`213`	`213`	`default:`
`214`	`214`	`}`
`215`		`- if len(p) > 0xffff {`
`216`		`- return 0, &quic.DatagramTooLargeError{MaxDatagramPayloadSize: 0xffff}`
	`215`	`+ if len(p) > protocol.MaxUDPSize {`
	`216`	`+ return 0, &quic.DatagramTooLargeError{MaxDatagramPayloadSize: protocol.MaxUDPSize}`
`217`	`217`	`}`
`218`	`218`	`packetId := uint16(c.packetId.Add(1) % math.MaxUint16)`
`219`	`219`	`message := allocMessage()`