fix service memory leak
authorDave Watson <davejwatson@fb.com>
Thu, 19 Mar 2015 19:57:13 +0000 (12:57 -0700)
committerNoam Lerner <noamler@fb.com>
Wed, 25 Mar 2015 22:34:34 +0000 (15:34 -0700)
commit3e3b5e3566a9036fd86525d62a9ad110e72861b7
treea14a34adbe59f940ebc7b4135ddfdf040718e6e2
parentdb66df3bcee0c6b8fdc9f94879efa805e4001d83
fix service memory leak

Summary:
This code is only used in a test so far.

Basically it looks like Pipeline has some dependency issues - any addBack() pipelines have to be deleted *after* the main pipeline is destroyed.  Ideally if the pipeline is already closed, the destruction order wouldn't matter.  There is currently no removeBack() call either.   This is a quick fix for the test to just delete stuff in the right order, we can discuss a better solution when @jsedgwick returns

Test Plan: fbconfig --sanitize=address --with-project-version=clang:3.5 --clang folly/wangle/service && fbmake runtests

Reviewed By: hans@fb.com

Subscribers: trunkagent, doug, fugalh, folly-diffs@, yfeldblum, jsedgwick

FB internal diff: D1903200

Tasks: 6415578

Signature: t1:1903200:1426528298:e109dcc0ec586a505a26cd95a6f20434d22cbd37
folly/wangle/bootstrap/ServerBootstrap-inl.h
folly/wangle/service/ServiceTest.cpp