diff options
author | Eric Laurent <elaurent@google.com> | 2014-02-20 16:26:11 -0800 |
---|---|---|
committer | Eric Laurent <elaurent@google.com> | 2014-02-20 16:26:11 -0800 |
commit | 8ea16e4b0a7d398d26887c18675b3899de5d779d (patch) | |
tree | 7bb7efa2106401a375cfe8874b6f4bb03dfdfb28 /services/audioflinger/AudioFlinger.h | |
parent | 56df9ff31d583ad3eae4f279a3df550273c58e1e (diff) | |
download | frameworks_av-8ea16e4b0a7d398d26887c18675b3899de5d779d.zip frameworks_av-8ea16e4b0a7d398d26887c18675b3899de5d779d.tar.gz frameworks_av-8ea16e4b0a7d398d26887c18675b3899de5d779d.tar.bz2 |
audioflinger: fix race condition in SyncEvent callback
Now that the SyncEvent callback is implemented by the
RecordTrack instead of the RecordThread, there is a possibility
that the callback is called after the track deletion.
SyncEvent callback now uses a weak pointer instead of
a raw pointer as cookie. This allows the callback implementer to
acquire a strong reference on the object pointed to by the cookie.
Bug: 13114128.
Change-Id: Id61b8f06044ed1e52c6f7e7c666cdede68340de2
Diffstat (limited to 'services/audioflinger/AudioFlinger.h')
-rw-r--r-- | services/audioflinger/AudioFlinger.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/services/audioflinger/AudioFlinger.h b/services/audioflinger/AudioFlinger.h index 4799beb..c5b8d33 100644 --- a/services/audioflinger/AudioFlinger.h +++ b/services/audioflinger/AudioFlinger.h @@ -253,7 +253,7 @@ public: int triggerSession, int listenerSession, sync_event_callback_t callBack, - void *cookie) + wp<RefBase> cookie) : mType(type), mTriggerSession(triggerSession), mListenerSession(listenerSession), mCallback(callBack), mCookie(cookie) {} @@ -266,14 +266,14 @@ public: AudioSystem::sync_event_t type() const { return mType; } int triggerSession() const { return mTriggerSession; } int listenerSession() const { return mListenerSession; } - void *cookie() const { return mCookie; } + wp<RefBase> cookie() const { return mCookie; } private: const AudioSystem::sync_event_t mType; const int mTriggerSession; const int mListenerSession; sync_event_callback_t mCallback; - void * const mCookie; + const wp<RefBase> mCookie; mutable Mutex mLock; }; @@ -281,7 +281,7 @@ public: int triggerSession, int listenerSession, sync_event_callback_t callBack, - void *cookie); + wp<RefBase> cookie); private: class AudioHwDevice; // fwd declaration for findSuitableHwDev_l |