Return buffers to pool on network errors or queue overflow (#2609)

Use of sync.Pool requires a .Put for every .Get.
On success, the downstream ThriftProcessor invoked .Put via .DataRecd
but in error cases there seems to be a leak.

Signed-off-by: Carl Henrik Lunde <chlunde@ifi.uio.no>

cmd/agent/app/servers/tbuffered_server.go

@@ -98,9 +98,11 @@ func (s *TBufferedServer) Serve() {
 				s.metrics.PacketsProcessed.Inc(1)
 				s.updateQueueSize(1)
 			default:
+				s.readBufPool.Put(readBuf)
 				s.metrics.PacketsDropped.Inc(1)
 			}
 		} else {
+			s.readBufPool.Put(readBuf)
 			s.metrics.ReadError.Inc(1)
 		}
 	}