diff options
| author | gVisor bot <gvisor-bot@google.com> | 2020-04-20 14:31:12 -0700 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2020-04-20 14:31:12 -0700 |
| commit | 07b1b4cc98552a7d45d673ce1e4d1222bbffca7d (patch) | |
| tree | 0c09a287bf7c712e315e2dd2a49b8785054de986 /pkg/tcpip/transport/udp/protocol.go | |
| parent | 470633d7e916e7956f4ebd75559f92cf12067cbf (diff) | |
| parent | a551add5d8a5bf631cd9859c761e579fdb33ec82 (diff) | |
Merge pull request #2313 from kevinGC:firstn
PiperOrigin-RevId: 307477185
Diffstat (limited to 'pkg/tcpip/transport/udp/protocol.go')
| -rw-r--r-- | pkg/tcpip/transport/udp/protocol.go | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/pkg/tcpip/transport/udp/protocol.go b/pkg/tcpip/transport/udp/protocol.go index 6e31a9bac..52af6de22 100644 --- a/pkg/tcpip/transport/udp/protocol.go +++ b/pkg/tcpip/transport/udp/protocol.go @@ -68,8 +68,13 @@ func (*protocol) ParsePorts(v buffer.View) (src, dst uint16, err *tcpip.Error) { // that don't match any existing endpoint. func (p *protocol) HandleUnknownDestinationPacket(r *stack.Route, id stack.TransportEndpointID, pkt stack.PacketBuffer) bool { // Get the header then trim it from the view. - hdr := header.UDP(pkt.Data.First()) - if int(hdr.Length()) > pkt.Data.Size() { + h, ok := pkt.Data.PullUp(header.UDPMinimumSize) + if !ok { + // Malformed packet. + r.Stack().Stats().UDP.MalformedPacketsReceived.Increment() + return true + } + if int(header.UDP(h).Length()) > pkt.Data.Size() { // Malformed packet. r.Stack().Stats().UDP.MalformedPacketsReceived.Increment() return true |