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

Commit b291d5d5 authored by Sivan Reinstein's avatar Sivan Reinstein
Browse files

msm: ipa: Remove double call for mutex_unlock 



In case of an error in ipa_flt when deleting/modifying
rules, there's an uneccessary call to mutex_unlock once
commit fails.

CRs-Fixed: 681621
Change-Id: Icc147b4dd4d9bd379b76925544938c34dd443465
Acked-by: default avatarDavid Arinzon <darinzon@qti.qualcomm.com>
Signed-off-by: default avatarSivan Reinstein <sivanr@codeaurora.org>
parent d81481d4

drivers/platform/msm/ipa/ipa_flt.c

+0 −2
Original line number Diff line number Diff line
@@ -1255,7 +1255,6 @@ int ipa_del_flt_rule(struct ipa_ioc_del_flt_rule *hdls) 


	if (hdls->commit)

		if (ipa_ctx->ctrl->ipa_commit_flt(hdls->ip)) {

			mutex_unlock(&ipa_ctx->lock);

			result = -EPERM;

			goto bail;

		}
@@ -1297,7 +1296,6 @@ int ipa_mdfy_flt_rule(struct ipa_ioc_mdfy_flt_rule *hdls) 


	if (hdls->commit)

		if (ipa_ctx->ctrl->ipa_commit_flt(hdls->ip)) {

			mutex_unlock(&ipa_ctx->lock);

			result = -EPERM;

			goto bail;

		}