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

Commit 237a624f 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 154a7fbd

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()

{