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

Commit 0c39abc4 authored by rago's avatar rago
Browse files

Fix potential NULL dereference in Visualizer effect 

Bug: 30229821

Test: fixing CL. Existing unit tests still pass.

Change-Id: I73d3fe413dd7590470c5b3b19569d6a7969c4dd8
parent 57ac6634

visualizer/offload_visualizer.c

+7 −3
Original line number Diff line number Diff line
@@ -826,9 +826,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)) {

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

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

            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;

        }