Commit 5ac71711 authored Oct 05, 2018 by Darshan Kumsi Srinivasa Committed by Gerrit - the friendly Code Review server Jan 06, 2019

msm: vidc: ignore processing responses in invalid state



No need to process response messages from video hardware
after device went into invalid state. Processing responses
may result in use-after-free memory fault because client
might free all the resources after error.

Change-Id: I3bfb26e5aa52aba33b7b62cda7820dcbc5fe033f
Signed-off-by: Darshan Kumsi Srinivasa <darssr@codeaurora.org>

parent 8e1e1a68

drivers/media/platform/msm/vidc/venus_hfi.c

+6 −1

Original line number	Diff line number	Diff line
		@@ -3689,7 +3689,12 @@ err_no_work:
		for (i = 0; !IS_ERR_OR_NULL(device->response_pkt) &&
		i < num_responses; ++i) {
		struct msm_vidc_cb_info *r = &device->response_pkt[i];

		if (!__core_in_valid_state(device)) {
		dprintk(VIDC_ERR,
		"Ignore responses from %d to %d as device is in invalid state",
		(i + 1), num_responses);
		break;
		}
		device->callback(r->response_type, &r->response);
		}