diff options
| author | Zhou Song <zhous@codeaurora.org> | 2016-03-22 16:13:09 +0800 |
|---|---|---|
| committer | Steve Kondik <steve@cyngn.com> | 2016-04-13 13:54:56 -0700 |
| commit | 5153cdc35e816a2224da1dac0e4d316bf847473d (patch) | |
| tree | 9734ec6250a5acab21974cc155e9f0d32cacdb1d /media/libmedia/AudioTrack.cpp | |
| parent | 20f078273de062f24b603e12197667b47ab20d05 (diff) | |
| download | frameworks_av-5153cdc35e816a2224da1dac0e4d316bf847473d.zip frameworks_av-5153cdc35e816a2224da1dac0e4d316bf847473d.tar.gz frameworks_av-5153cdc35e816a2224da1dac0e4d316bf847473d.tar.bz2 | |
libmedia: Initialize mState/mActive value to avoid possible crash
If track creation fails on server side, mState/mActive is left
unitialized. After this a crash can happen when track stop()
is called because of a reference to a NULL object.
Initialize mState/mActive to specific value in initialized list
of track constructor.
Change-Id: If8c3611e3229c0c1b14b81285e07b9357fec7658
CRs-Fixed: 992608
Diffstat (limited to 'media/libmedia/AudioTrack.cpp')
| -rw-r--r-- | media/libmedia/AudioTrack.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/media/libmedia/AudioTrack.cpp b/media/libmedia/AudioTrack.cpp index d7256f8..ae016ef 100644 --- a/media/libmedia/AudioTrack.cpp +++ b/media/libmedia/AudioTrack.cpp @@ -165,6 +165,7 @@ status_t AudioTrack::getMinFrameCount( AudioTrack::AudioTrack() : mStatus(NO_INIT), + mState(STATE_STOPPED), mIsTimed(false), mPreviousPriority(ANDROID_PRIORITY_NORMAL), mPreviousSchedulingGroup(SP_DEFAULT), @@ -196,6 +197,7 @@ AudioTrack::AudioTrack( const audio_attributes_t* pAttributes, bool doNotReconnect) : mStatus(NO_INIT), + mState(STATE_STOPPED), mIsTimed(false), mPreviousPriority(ANDROID_PRIORITY_NORMAL), mPreviousSchedulingGroup(SP_DEFAULT), @@ -227,6 +229,7 @@ AudioTrack::AudioTrack( const audio_attributes_t* pAttributes, bool doNotReconnect) : mStatus(NO_INIT), + mState(STATE_STOPPED), mIsTimed(false), mPreviousPriority(ANDROID_PRIORITY_NORMAL), mPreviousSchedulingGroup(SP_DEFAULT), @@ -478,7 +481,6 @@ status_t AudioTrack::set( } mStatus = NO_ERROR; - mState = STATE_STOPPED; mUserData = user; mLoopCount = 0; mLoopStart = 0; |