Skip to content

Commit 74b84b8

Browse files
committed
packet: fix misusing of defer
defer capture arguments at once, so modify arguments after defer line not works
1 parent 1c55c36 commit 74b84b8

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

packet/conn.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,9 @@ func (c *Conn) ReadPacket() ([]byte, error) {
9090
func (c *Conn) ReadPacketReuseMem(dst []byte) ([]byte, error) {
9191
// Here we use `sync.Pool` to avoid allocate/destroy buffers frequently.
9292
buf := utils.BytesBufferGet()
93-
defer utils.BytesBufferPut(buf)
93+
defer func() {
94+
utils.BytesBufferPut(buf)
95+
}()
9496

9597
if err := c.ReadPacketTo(buf); err != nil {
9698
return nil, errors.Trace(err)

0 commit comments

Comments
 (0)