commit dcfefde6be8f9f33bbd2b3546f3be946d958d677
author beanz <beanz@marvell.com> Mon Nov 05 09:51:43 2012 +0800
committer beanz <beanz@marvell.com> Mon Nov 05 09:51:43 2012 +0800
tree fc085ca88168b58486f942b64516a8008cfe1545
parent 1baa01a52f24566c134313fade8b19a582654638

fix a race condition of MediaPlayerServce::Client::mClient

there will be race condition between mClient.clear in MediaPlayerServce::Client::disconnect method
and c=client->mClient in in MediaPlayerServce::Client::notify method

Change-Id: I767294c4cb864c08e9c0e9f23d6d304a39016205
Signed-off-by: beanz <beanz@marvell.com>

media/libmediaplayerservice/MediaPlayerService.cpp

diff --git a/media/libmediaplayerservice/MediaPlayerService.cpp b/media/libmediaplayerservice/MediaPlayerService.cpp
index c47fbd6..3a4122a 100644
--- a/media/libmediaplayerservice/MediaPlayerService.cpp
+++ b/media/libmediaplayerservice/MediaPlayerService.cpp
@@ -523,8 +523,8 @@
     {
         Mutex::Autolock l(mLock);
         p = mPlayer;
+        mClient.clear();
     }
-    mClient.clear();
 
     mPlayer.clear();