summaryrefslogtreecommitdiffstats
path: root/media
diff options
context:
space:
mode:
authorEric Laurent <elaurent@google.com>2013-03-26 12:25:30 -0700
committerEric Laurent <elaurent@google.com>2013-03-26 12:25:30 -0700
commitdb372594bf27390d2bee089e35c5fb8f67370026 (patch)
treef1f73f399cab41da6a204177610f341e0157a63e /media
parent190cdbab6ba24519d6b5e8bec6c2c74e6650e284 (diff)
downloadframeworks_av-db372594bf27390d2bee089e35c5fb8f67370026.zip
frameworks_av-db372594bf27390d2bee089e35c5fb8f67370026.tar.gz
frameworks_av-db372594bf27390d2bee089e35c5fb8f67370026.tar.bz2
ToneGenerator: fix AudioTrack pointer init
The pointer to AudioTrack should be initialized before early return from ToneGenerator constructor because it is tested by the destructor. Bug 8140963 Change-Id: I9a7dfb60ba162b75dfaa5630ab7fc9485afd0074
Diffstat (limited to 'media')
-rw-r--r--media/libmedia/ToneGenerator.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/media/libmedia/ToneGenerator.cpp b/media/libmedia/ToneGenerator.cpp
index 1c0268f..58d495e 100644
--- a/media/libmedia/ToneGenerator.cpp
+++ b/media/libmedia/ToneGenerator.cpp
@@ -803,6 +803,7 @@ ToneGenerator::ToneGenerator(audio_stream_type_t streamType, float volume, bool
ALOGV("ToneGenerator constructor: streamType=%d, volume=%f", streamType, volume);
mState = TONE_IDLE;
+ mpAudioTrack = NULL;
if (AudioSystem::getOutputSamplingRate(&mSamplingRate, streamType) != NO_ERROR) {
ALOGE("Unable to marshal AudioFlinger");
@@ -811,7 +812,6 @@ ToneGenerator::ToneGenerator(audio_stream_type_t streamType, float volume, bool
mThreadCanCallJava = threadCanCallJava;
mStreamType = streamType;
mVolume = volume;
- mpAudioTrack = NULL;
mpToneDesc = NULL;
mpNewToneDesc = NULL;
// Generate tone by chunks of 20 ms to keep cadencing precision