Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b3719251 authored by Glenn Kasten's avatar Glenn Kasten
Browse files

deleteTrackName now asserts on bad input parameter 

This is safe, as the input parameter is always track->name(),
which must be valid.

Change-Id: Iea8ea3a5706c27026335526ba8851030d00681f8
parent aba0568d

services/audioflinger/AudioMixer.cpp

+16 −17
Original line number Diff line number Diff line
@@ -124,7 +124,7 @@ void AudioMixer::invalidateState(uint32_t mask) 
void AudioMixer::deleteTrackName(int name)

{

    name -= TRACK0;

    if (uint32_t(name) < MAX_NUM_TRACKS) {

    assert(uint32_t(name) < MAX_NUM_TRACKS);

    ALOGV("deleteTrackName(%d)", name);

    track_t& track(mState.tracks[ name ]);

    if (track.enabled != 0) {
@@ -142,7 +142,6 @@ void AudioMixer::deleteTrackName(int name) 
    track.volumeInc[1] = 0;

    mTrackNames &= ~(1<<name);

}

}



void AudioMixer::enable()

{