| commit | c1dbd7855353cb0d148e7217ddccd9aa960bae57 | [log] [tgz] |
|---|---|---|
| author | Shunkai Yao <yaoshunkai@google.com> | Tue Nov 07 00:38:34 2023 +0000 |
| committer | Android Build Cherrypicker Worker <android-build-cherrypicker-worker@google.com> | Tue Nov 07 00:38:34 2023 +0000 |
| tree | 64682798b9cdee832698065eba5846668bd0761c | |
| parent | 6d6a821f49ea48bb8119b2509912d1d5f018e1fe [diff] [blame] |
Validate IAfEffectHandle before access virtual methods Bug: 290218127 Test: atest DeviceEffectTest#testIsEffectSupportedForDevice --iterations 50 (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:8ade257ef06ec593f0bf17440cbf57bdd591d8b1) Merged-In: I2e427a80e19c2e5987673bfd014a5da5566f4a24 Change-Id: I2e427a80e19c2e5987673bfd014a5da5566f4a24
diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp index a277ad1..29c6845 100644 --- a/services/audioflinger/AudioFlinger.cpp +++ b/services/audioflinger/AudioFlinger.cpp
@@ -4270,7 +4270,7 @@ response->id = idOut; response->enabled = enabledOut != 0; - response->effect = handle->asIEffect(); + response->effect = handle.get() ? handle->asIEffect() : nullptr; response->desc = VALUE_OR_RETURN_STATUS( legacy2aidl_effect_descriptor_t_EffectDescriptor(descOut));