From 15eb7a6fa9828fb7aed5ea58529763806d1a7c86 Mon Sep 17 00:00:00 2001 From: Mikhail Naganov Date: Wed, 10 Oct 2018 14:40:54 -0700 Subject: [PATCH] audio: Add a missing break to EffectsFactory::getAllDescriptors Would the legacy HAL return -ENOENT while retrieving an effect, the enumeration has to be aborted, and the effects queried so far need to be returned. A missing 'break' was causing returning no effects at all. Bug: 117567478 Test: make Change-Id: I7cc05004f88b39272d06d2ed33a0693f8e1e3d04 Merged-In: I7cc05004f88b39272d06d2ed33a0693f8e1e3d04 --- .../include/effect/all-versions/default/EffectsFactory.impl.h | 1 + 1 file changed, 1 insertion(+) diff --git a/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h b/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h index 1882a2c490..b0351c968f 100644 --- a/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h +++ b/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h @@ -101,6 +101,7 @@ restart: case -ENOENT: { // No more effects available. result.resize(i); + break; } default: { result.resize(0);