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

Commit 8d857508 authored by rago's avatar rago Committed by Gerrit - the friendly Code Review server
Browse files

Fix security vulnerability: Equalizer command might allow negative indexes 

Bug: 32247948
Bug: 32438598
Bug: 32436341

Test: use POC on bug or cts security test

Change-Id: I56a92582687599b5b313dea1abcb8bcb19c7fc0e
(cherry picked from commit 3f37d4ef)
(cherry picked from commit ceb7b2d7)
(cherry picked from commit d72ea85c)
parent a5fa60b7

post_proc/equalizer.c

+16 −4
Original line number Diff line number Diff line
@@ -274,8 +274,12 @@ int equalizer_get_parameter(effect_context_t *context, effect_param_t *p, 


    case EQ_PARAM_BAND_LEVEL:

        param2 = *param_tmp;

        if (param2 >= NUM_EQ_BANDS) {

        if (param2 < 0 || param2 >= NUM_EQ_BANDS) {

            p->status = -EINVAL;

            if (param2 < 0) {

                android_errorWriteLog(0x534e4554, "32438598");

                ALOGW("\tERROR EQ_PARAM_BAND_LEVEL band %d", param2);

            }

            break;

        }

        *(int16_t *)value = (int16_t)equalizer_get_band_level(eq_ctxt, param2);
@@ -283,8 +287,12 @@ int equalizer_get_parameter(effect_context_t *context, effect_param_t *p, 


    case EQ_PARAM_CENTER_FREQ:

        param2 = *param_tmp;

        if (param2 >= NUM_EQ_BANDS) {

        if (param2 < 0 || param2 >= NUM_EQ_BANDS) {

            p->status = -EINVAL;

            if (param2 < 0) {

                android_errorWriteLog(0x534e4554, "32436341");

                ALOGW("\tERROR EQ_PARAM_CENTER_FREQ band %d", param2);

            }

            break;

        }

        *(int32_t *)value = equalizer_get_center_frequency(eq_ctxt, param2);
@@ -292,8 +300,12 @@ int equalizer_get_parameter(effect_context_t *context, effect_param_t *p, 


    case EQ_PARAM_BAND_FREQ_RANGE:

        param2 = *param_tmp;

        if (param2 >= NUM_EQ_BANDS) {

        if (param2 < 0 || param2 >= NUM_EQ_BANDS) {

            p->status = -EINVAL;

            if (param2 < 0) {

                android_errorWriteLog(0x534e4554, "32247948");

                ALOGW("\tERROR EQ_PARAM_BAND_FREQ_RANGE band %d", param2);

            }

           break;

        }

       equalizer_get_band_freq_range(eq_ctxt, param2, (uint32_t *)value,