From f6e8c701024f6c5dac552e794dcc61016a7f1bf7 Mon Sep 17 00:00:00 2001 From: Sharad Sangle Date: Thu, 24 Dec 2015 19:58:54 +0530 Subject: AudioMixer: delete reformatBuffer provider in proper order If mixer is creating reformatBufferProvider and downMixerBufferProvider while adding a track it first creates downMixerProvider then reformatBufferProvider. While deleting track it first deletes downMixerBufferProvider and then reformatBufferProvider, which should be in reverse order, the object created last should be deleted first. Change-Id: I844e7862280fe37c3167b31e92bbb27aa9463315 --- services/audioflinger/AudioMixer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'services/audioflinger') diff --git a/services/audioflinger/AudioMixer.cpp b/services/audioflinger/AudioMixer.cpp index 27a2f65..b830985 100644 --- a/services/audioflinger/AudioMixer.cpp +++ b/services/audioflinger/AudioMixer.cpp @@ -445,10 +445,10 @@ void AudioMixer::deleteTrackName(int name) // delete the resampler delete track.resampler; track.resampler = NULL; - // delete the downmixer - mState.tracks[name].unprepareForDownmix(); // delete the reformatter mState.tracks[name].unprepareForReformat(); + // delete the downmixer + mState.tracks[name].unprepareForDownmix(); // delete the timestretch provider delete track.mTimestretchBufferProvider; track.mTimestretchBufferProvider = NULL; -- cgit v1.1