diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2014-12-19 15:15:50 +0100 |
|---|---|---|
| committer | Eric Laurent <elaurent@google.com> | 2015-04-24 13:39:13 -0700 |
| commit | 09af813fef22c2c3cbb34795e290ebab3db65e35 (patch) | |
| tree | 32d50d5c19355fc9bf202c5747285d7baa919452 | |
| parent | 6bd9fcf524e77a1ca7469d6d91c58afd242d388a (diff) | |
| download | external_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>
| -rw-r--r-- | test/test-platform/TestPlatform.cpp | 3 |
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); |