Merge "visualizer: fix issue of subtraction overflow" (e6a167ec) · Commits · e / os / android_hardware_qcom_audio

visualizer/offload_visualizer.c

+6 −3

Original line number	Diff line number	Diff line
		@@ -835,13 +835,16 @@ int visualizer_command(effect_context_t * context, uint32_t cmdCode, uint32_t cm
		if (context->state == EFFECT_STATE_ACTIVE) {
		int32_t latency_ms = visu_ctxt->latency;
		const uint32_t delta_ms = visualizer_get_delta_time_ms_from_updated_time(visu_ctxt);
		latency_ms -= delta_ms;
		if (latency_ms < 0) {
		if (latency_ms < delta_ms) {
		latency_ms = 0;
		} else {
		latency_ms -= delta_ms;
		}
		const uint32_t delta_smp = context->config.inputCfg.samplingRate * latency_ms / 1000;

		int32_t capture_point = visu_ctxt->capture_idx - visu_ctxt->capture_size - delta_smp;
		int32_t capture_point = 0;
		__builtin_sub_overflow(visu_ctxt->capture_idx, visu_ctxt->capture_size + delta_smp,
		&capture_point);
		int32_t capture_size = visu_ctxt->capture_size;
		if (capture_point < 0) {
		int32_t size = -capture_point;