Fix Fiber::LocalData leak for large fiber locals

Summary: facepaw

Reviewed By: yfeldblum

Differential Revision: D4856227

fbshipit-source-id: 17a73d4d630c6efe1763bbd8aff8b7a2c7721c27

diff --git a/folly/fibers/Fiber.cpp b/folly/fibers/Fiber.cpp
index 39dcc3332983d1bde8e0867d05d53d2d1599c8f9..f5995de31b2a3c5618b68b32425ee652664b8321 100644 (file)
--- a/folly/fibers/Fiber.cpp
+++ b/folly/fibers/Fiber.cpp
@@ -189,6 +189,10 @@ void Fiber::preempt(State state) {
   }
 }
 
+Fiber::LocalData::~LocalData() {
+  reset();
+}
+
 Fiber::LocalData::LocalData(const LocalData& other) : data_(nullptr) {
   *this = other;
 }

diff --git a/folly/fibers/Fiber.h b/folly/fibers/Fiber.h
index 0fcd903d164057cef7cc491bafcbd71d5e41997f..36af25ebb0862c2636480c61d96fc7c62c3d4352 100644 (file)
--- a/folly/fibers/Fiber.h
+++ b/folly/fibers/Fiber.h
@@ -131,6 +131,7 @@ class Fiber {
   class LocalData {
    public:
     LocalData() {}
+    ~LocalData();
     LocalData(const LocalData& other);
     LocalData& operator=(const LocalData& other);