MB-20585: Fix three memory leaks 13/66913/2
authorJim Walker <jim@couchbase.com>
Fri, 19 Aug 2016 13:28:39 +0000 (13:28 +0000)
committerDaniel Owen <owend@couchbase.com>
Mon, 22 Aug 2016 10:44:37 +0000 (10:44 +0000)
commita288f6c6b3e8bc15cecd969404545f58cdf3da81
tree98e7f86707d7389bb512cffdcd89f10da49be9b7
parent52311e8dc0526c5c0029f3599b1d7cbbb993bcf8
MB-20585: Fix three memory leaks

Recent changes have snuck some leaks in.

1) CouchKVStore refactoring allocated two buffers when a single
larger buffer should be used. The second buffer never gets freed.
Also a new/free mistmatch between couchstore/ep-engine is fixed.

Introduced by - 8350175

2) The watson -> master merge fixed a TSAN issue, but introduced a leak
in DcpConsumer code.

Introduced by - 7ed33aa

3) Changes in connmap mean test code can now leak cookies because the
cleanup task runs every second allowing a test to finish with an
allocated cookie.

Introduced by dd2bf1e

Change-Id: I8e63a51a4a6ec0c88782a6f487ced648d36afd2e
Reviewed-on: http://review.couchbase.org/66913
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
src/couch-kvstore/couch-kvstore.cc
src/dcp/dcpconnmap.cc
src/dcp/stream.cc