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

Commit 5437fc31 authored by Mohammed Javid's avatar Mohammed Javid
Browse files

msm: ipa: Fix to validate the user inputs 



Adding code changes to validate user inputs
if filter_index_list_len value is zero case.
Because of recent fix I989eed2e0 these changes
are required.

Change-Id: Ica1e8c4dde96d010b570abcaf85bac31345e8d0c
Acked-by: default avatarAshok Vuyyuru <avuyyuru@qti.qualcomm.com>
Signed-off-by: default avatarMohammed Javid <mjavid@codeaurora.org>
parent 9c8919d1

drivers/platform/msm/ipa/ipa_v2/ipa_qmi_service.c

+3 −1
Original line number Diff line number Diff line
@@ -706,7 +706,9 @@ int qmi_filter_notify_send(struct ipa_fltr_installed_notif_req_msg_v01 *req) 
				req->filter_index_list[i].filter_handle,

				req->filter_index_list[i].filter_index);

		return -EINVAL;

	} else if (req->install_status != IPA_QMI_RESULT_SUCCESS_V01) {

	}



	if (req->install_status != IPA_QMI_RESULT_SUCCESS_V01) {

		IPAWANERR(" UL filter rule for pipe %d install_status = %d\n",

			req->source_pipe_index, req->install_status);

		return -EINVAL;

drivers/platform/msm/ipa/ipa_v3/ipa_qmi_service.c

+4 −2
Original line number Diff line number Diff line
@@ -610,7 +610,7 @@ int ipa3_qmi_filter_request_send(struct ipa_install_fltr_rule_req_msg_v01 *req) 
		IPAWANDBG("IPACM pass zero rules to Q6\n");

	} else {

		IPAWANDBG("IPACM pass %u rules to Q6\n",

		req->filter_spec_ex_list_len);

		req->filter_spec_list_len);

	}



	if (req->filter_spec_list_len >= QMI_IPA_MAX_FILTERS_V01) {
@@ -919,7 +919,9 @@ int ipa3_qmi_filter_notify_send( 
		req->source_pipe_index,

		req->rule_id_len);

		return -EINVAL;

	} else if (req->install_status != IPA_QMI_RESULT_SUCCESS_V01) {

	}



	if (req->install_status != IPA_QMI_RESULT_SUCCESS_V01) {

		IPAWANERR(" UL filter rule for pipe %d install_status = %d\n",

			req->source_pipe_index, req->install_status);

		return -EINVAL;