From c2bd153760b0ae24b1cd0183741742ae82f4c4da Mon Sep 17 00:00:00 2001 From: Kevin Krakauer Date: Tue, 29 Mar 2022 17:45:26 -0700 Subject: [PATCH] don't check refcount in packet buffer's destroy function The function is only called when refcount is zero, so we don't have to check again. PiperOrigin-RevId: 438171100 --- pkg/tcpip/stack/packet_buffer.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/pkg/tcpip/stack/packet_buffer.go b/pkg/tcpip/stack/packet_buffer.go index 8e5293568..4ec1ad5ee 100644 --- a/pkg/tcpip/stack/packet_buffer.go +++ b/pkg/tcpip/stack/packet_buffer.go @@ -188,9 +188,7 @@ func NewPacketBuffer(opts PacketBufferOptions) *PacketBuffer { // pool. func (pk *PacketBuffer) DecRef() { pk.packetBufferRefs.DecRef(func() { - if pk.packetBufferRefs.refCount == 0 { - pkPool.Put(pk) - } + pkPool.Put(pk) }) }