aboutsummaryrefslogtreecommitdiffstats
path: root/test/test-platform
diff options
context:
space:
mode:
authorKevin Rocard <kevinx.rocard@intel.com>2014-12-19 15:15:50 +0100
committerEric Laurent <elaurent@google.com>2015-04-24 13:39:13 -0700
commit09af813fef22c2c3cbb34795e290ebab3db65e35 (patch)
tree32d50d5c19355fc9bf202c5747285d7baa919452 /test/test-platform
parent6bd9fcf524e77a1ca7469d6d91c58afd242d388a (diff)
downloadexternal_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.zip
external_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.tar.gz
external_parameter-framework-09af813fef22c2c3cbb34795e290ebab3db65e35.tar.bz2
Remove explicit server stop
Stopping the remote-processor in the stop command has two side effect: - stopping the remote-processor thread will fail as it is not possible to join a thread to itself. This leads to the leak of all the thread object (destructor not called). - the server will not return the exit command status as it has been stopped during it's execution. Remove the remote-processor stop as it will be called during it's destruction. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
Diffstat (limited to 'test/test-platform')
-rw-r--r--test/test-platform/TestPlatform.cpp3
1 files changed, 0 insertions, 3 deletions
diff --git a/test/test-platform/TestPlatform.cpp b/test/test-platform/TestPlatform.cpp
index a6613e9..9cb45f7 100644
--- a/test/test-platform/TestPlatform.cpp
+++ b/test/test-platform/TestPlatform.cpp
@@ -154,9 +154,6 @@ CTestPlatform::CommandReturn CTestPlatform::exit(
{
(void)remoteCommand;
- // Stop local server
- _pRemoteProcessorServer->stop();
-
// Release the main blocking semaphore to quit application
sem_post(&_exitSemaphore);