diff options
| author | Eric Laurent <elaurent@google.com> | 2013-11-15 12:02:28 -0800 |
|---|---|---|
| committer | Eric Laurent <elaurent@google.com> | 2013-11-15 12:11:39 -0800 |
| commit | d7e59228caad3867794d847f6bf163c6495e9506 (patch) | |
| tree | f09784ce425a6928c56a932101752b63b21d8f44 /services/audioflinger/Threads.h | |
| parent | 7dae71d606ded1dbc2aa9733c3d98ffac57988f2 (diff) | |
| download | frameworks_av-d7e59228caad3867794d847f6bf163c6495e9506.zip frameworks_av-d7e59228caad3867794d847f6bf163c6495e9506.tar.gz frameworks_av-d7e59228caad3867794d847f6bf163c6495e9506.tar.bz2 | |
audioflinger: do not use raw pointer for tracks
Commit 9da3d95 surfaced a problem caused by the use of a raw
pointer to a track in offload thread implementation.
Pointers to tracks should always be weak or strong pointers.
Bug: 11708529.
Change-Id: Ic48632532d186c9be8261f73cefdf824b9fbbd2b
Diffstat (limited to 'services/audioflinger/Threads.h')
| -rw-r--r-- | services/audioflinger/Threads.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/services/audioflinger/Threads.h b/services/audioflinger/Threads.h index a0b53cb..207f1eb 100644 --- a/services/audioflinger/Threads.h +++ b/services/audioflinger/Threads.h @@ -762,7 +762,7 @@ private: bool mFlushPending; size_t mPausedWriteLength; // length in bytes of write interrupted by pause size_t mPausedBytesRemaining; // bytes still waiting in mixbuffer after resume - Track *mPreviousTrack; // used to detect track switch + wp<Track> mPreviousTrack; // used to detect track switch }; class AsyncCallbackThread : public Thread { |