Release resources in TaskContext before notifying user.

Resolves: OLPEDGE-1005

Signed-off-by: Serhii Lozynskyi <[email protected]>

Author: meravingen123

olp-cpp-sdk-core/include/olp/core/client/TaskContext.h

@@ -22,11 +22,11 @@
 #include <atomic>
 #include <mutex>
 
+#include <olp/core/client/ApiError.h>
+#include <olp/core/client/ApiResponse.h>
+#include <olp/core/client/CancellationContext.h>
+#include <olp/core/client/CancellationToken.h>
 #include <olp/core/client/Condition.h>
-#include "olp/core/client/ApiError.h"
-#include "olp/core/client/ApiResponse.h"
-#include "olp/core/client/CancellationContext.h"
-#include "olp/core/client/CancellationToken.h"
 
 namespace olp {
 namespace client {
@@ -161,6 +161,11 @@ class TaskContext {
         callback(std::move(user_response));
       }
 
+      // Resources needs to be released before notification, else lambas would
+      // have captured resources like network or TaskScheduler
+      function = nullptr;
+      callback = nullptr;
+
       condition_.Notify();
 
       state_.store(State::COMPLETED);