Merge "Add EFFECT_CMD_SET_PARAM parameter checking to Preset Reverb" into lmp-mr1-dev am: 31b0c9cb33 am: be30bb2645 am: e2d671ce19 am: 8aa73cce69 am: 3785a50ef4 am: a060ca58ff am: dee287e40c am: 9dc85bf51e am: 59e93ab3ab am: 3b564f6e42 am: 62e3e59029

am: 26214cfbaa Change-Id: Ifd11bdfbb52cab13bad5379e0c8c1a696094f054
7 years ago · 1fee4c334c
parent a307668b1a 26214cfbaa
commit 1fee4c334c
1 changed files with 4 additions and 0 deletions
--- a/media/libeffects/lvm/wrapper/Reverb/EffectReverb.cpp
+++ b/media/libeffects/lvm/wrapper/Reverb/EffectReverb.cpp
@ -1889,6 +1889,10 @@ int Reverb_setParameter (ReverbContext *pContext, void *pParam, void *pValue, in
        if (param != REVERB_PARAM_PRESET) {
            return -EINVAL;
        }
+        if (vsize < (int)sizeof(uint16_t)) {
+            android_errorWriteLog(0x534e4554, "67647856");
+            return -EINVAL;
+        }

        uint16_t preset = *(uint16_t *)pValue;
        ALOGV("set REVERB_PARAM_PRESET, preset %d", preset);