Don't bind process to make task for pushing thread pool

Sometimes crash occured on dtor of binded std::function after moved to
thread pool task queue. We can avoid it by using inbuf as shared ptr so
as to being delivered into lamda capture.

Change-Id: I76d68d91ca75bdc0c37bfb5b125a1e2956744010
Signed-off-by: Kyungwook Tak <k.tak@samsung.com>

diff --git a/src/framework/service/server-service.cpp b/src/framework/service/server-service.cpp
index 3d66ce3..70c1bdc 100644 (file)
--- a/src/framework/service/server-service.cpp
+++ b/src/framework/service/server-service.cpp
@@ -358,16 +358,15 @@ void ServerService::onMessageProcess(const ConnShPtr &connection)
                ThrowExc(InternalError, "Message from unknown sock id");
        }
 
-       auto inbuf = connection->receive();
-       auto task = [this, &connection, process](RawBuffer &buffer) {
-               auto outbuf = (*process)(connection, buffer);
+       auto inbufPtr = std::make_shared<RawBuffer>(connection->receive());
+
+       m_workqueue.submit([this, &connection, process, inbufPtr]() {
+               auto outbuf = (*process)(connection, *inbufPtr);
 
                connection->send(outbuf);
 
                CpuUsageManager::reset();
-       };
-
-       m_workqueue.submit(std::bind(task, std::move(inbuf)));
+       });
 }
 
 }