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

Commit de0168ba authored by Ricardo Garcia's avatar Ricardo Garcia Committed by Android (Google) Code Review
Browse files

Merge "Fix potential NULL dereference in Visualizer effect"

parents bf486d07 314274df

visualizer/offload_visualizer.c

+7 −3
Original line number Diff line number Diff line
@@ -869,9 +869,13 @@ int visualizer_command(effect_context_t * context, uint32_t cmdCode, uint32_t cm 
    case VISUALIZER_CMD_MEASURE: {

        if (pReplyData == NULL || replySize == NULL ||

                *replySize < (sizeof(int32_t) * MEASUREMENT_COUNT)) {

            if (replySize == NULL) {

                ALOGV("%s VISUALIZER_CMD_MEASURE error replySize NULL", __func__);

            } else {

                ALOGV("%s VISUALIZER_CMD_MEASURE error *replySize %u <"

                        "(sizeof(int32_t) * MEASUREMENT_COUNT) %zu",

                        __func__, *replySize, sizeof(int32_t) * MEASUREMENT_COUNT);

            }

            android_errorWriteLog(0x534e4554, "30229821");

            return -EINVAL;

        }