(wangle) fix after-delete assert
authorHans Fugal <fugalh@fb.com>
Tue, 21 Oct 2014 22:50:02 +0000 (15:50 -0700)
committerdcsommer <dcsommer@fb.com>
Wed, 29 Oct 2014 23:04:10 +0000 (16:04 -0700)
commitfd915b73606e09a5f46a1bca0a5d3643a1567014
tree9e303bfe0f63628783d5492f7ab8241e99f5e5a5
parent5aa44fb2070479b5bfd88d1e3534439f4d54e238
(wangle) fix after-delete assert

Summary:
This would cause debug builds to do a bad thing (access the variable `this->detached_` within an assert, after `delete this`).

Test Plan: unit tests

Hopefully now that we have a dummy cpp file in `folly/wangle/detail` contbuild
will pick it up and all the dependencies will also run their tests.
Right now, we suspect maybe maelstrom (@wez) and adinvoicer and zookeeper
(@jying) and probably others are seeing this in unit test failures (esp. if
they use asan, which is rightly detecting read after free). Hoping contbuild
will catch them.

Reviewed By: davejwatson@fb.com

Subscribers: net-systems@, fugalh, exa, njormrod, folly-diffs@, wez, dcapel, jying, cgheorghe

FB internal diff: D1630301

Tasks: 54245465435720

Blame Revision: D1618240
folly/wangle/detail/Core.h