| commit | b11b7fe1b13e8e0e98f376f28767bd2dee31beb7 | [log] [tgz] |
|---|---|---|
| author | Shunkai Yao <yaoshunkai@google.com> | Wed Sep 06 21:54:08 2023 +0000 |
| committer | Shunkai Yao <yaoshunkai@google.com> | Thu Sep 07 21:25:09 2023 +0000 |
| tree | 2970bd02472e5a12883ebff1a0969c24a511127e | |
| parent | 6999bc03e98ddf2950ee1b5485c4c07ab871e74f [diff] |
Add AIDL union tag checking before access And improve some parameter logging Bug: 296954124 Test: Enable AIDL and test with YTM on Pixel Change-Id: Ic3373a138b8aa02d6c283342b48765ba909855de
diff --git a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionDynamicsProcessing.cpp b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionDynamicsProcessing.cpp index 89f8b83..f77c093 100644 --- a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionDynamicsProcessing.cpp +++ b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionDynamicsProcessing.cpp
@@ -227,7 +227,7 @@ RETURN_IF_ERROR(param.readFromValue(&enable)); return DynamicsProcessing::ChannelConfig( - {.enable = VALUE_OR_RETURN(convertIntegral<bool>(enable)), .channel = channel}); + {.channel = channel, .enable = VALUE_OR_RETURN(convertIntegral<bool>(enable))}); } ConversionResult<DynamicsProcessing::EqBandConfig>