mirrors/uquic
1
0
Fork 0
mirror of https://github.com/refraction-networking/uquic.git synced 2025-04-05 13:17:36 +03:00
Code Issues Projects Releases Packages Wiki Activity

introduce an assertion in the server

Browse source
This commit is contained in:
Marten Seemann 2020-06-29 10:25:55 +07:00
parent f926945ae5
commit 7bcb25f18d
2 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
server.go
View file

@ -322,7 +322,7 @@ func (s *baseServer) handlePacketImpl(p *receivedPacket) bool /* should the buff
} }
// Short header packets should never end up here in the first place // Short header packets should never end up here in the first place
if !hdr.IsLongHeader { if !hdr.IsLongHeader {
return false panic(fmt.Sprintf("misrouted packet: %#v", hdr))
} }
if hdr.Type == protocol.PacketTypeInitial && len(p.data) < protocol.MinInitialPacketSize { if hdr.Type == protocol.PacketTypeInitial && len(p.data) < protocol.MinInitialPacketSize {
s.logger.Debugf("Dropping a packet that is too small to be a valid Initial (%d bytes)", len(p.data)) s.logger.Debugf("Dropping a packet that is too small to be a valid Initial (%d bytes)", len(p.data))

1
server_test.go
View file

@ -234,6 +234,7 @@ var _ = Describe("Server", func() {
It("drops non-Initial packets", func() { It("drops non-Initial packets", func() {
serv.handlePacket(getPacket( serv.handlePacket(getPacket(
&wire.Header{ &wire.Header{
IsLongHeader: true,
Type: protocol.PacketTypeHandshake, Type: protocol.PacketTypeHandshake,
Version: serv.config.Versions[0], Version: serv.config.Versions[0],
}, },