On Thu, 05 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@xxxxxxxxx> wrote:
> Earlier, I responded to the wrong version of the patch :(
> Apologies.
>
>>-----Original Message-----
>>From: Sripada, Radhakrishna
>>Sent: Wednesday, April 4, 2018 4:00 PM
>>To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>>Cc: Sripada, Radhakrishna <radhakrishna.sripada@xxxxxxxxx>; Jani Nikula
>><jani.nikula@xxxxxxxxxxxxxxx>; Srivatsa, Anusha <anusha.srivatsa@xxxxxxxxx>
>>Subject: [PATCH v2] drm/i915: Fix memory leak in intel_hdcp auth
>>
>>Static code analysis tool reported memory leak in intel_hdcp_auth_downstream.
>>Fixing the memory leak.
>>
>>v2: Rebase, move free to a cleanup label(Jani)
>>
>>Cc: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>
>>Cc: Anusha Srivatsa <anusha.srivatsa@xxxxxxxxx>
>>Signed-off-by: Radhakrishna Sripada <radhakrishna.sripada@xxxxxxxxx>
> Reviewed-by: Anusha Srivatsa <anusha.srivatsa@xxxxxxxxx>
Thanks for the patch and the review, pushed to dinq.
BR,
Jani.
>
>>---
>> drivers/gpu/drm/i915/intel_hdcp.c | 9 ++++++---
>> 1 file changed, 6 insertions(+), 3 deletions(-)
>>
>>diff --git a/drivers/gpu/drm/i915/intel_hdcp.c
>>b/drivers/gpu/drm/i915/intel_hdcp.c
>>index 98a9c81e2dc1..2db5da550a1c 100644
>>--- a/drivers/gpu/drm/i915/intel_hdcp.c
>>+++ b/drivers/gpu/drm/i915/intel_hdcp.c
>>@@ -435,7 +435,7 @@ int intel_hdcp_auth_downstream(struct intel_digital_port
>>*intel_dig_port,
>>
>> ret = shim->read_ksv_fifo(intel_dig_port, num_downstream, ksv_fifo);
>> if (ret)
>>- return ret;
>>+ goto err;
>>
>> /*
>> * When V prime mismatches, DP Spec mandates re-read of @@ -451,12
>>+451,15 @@ int intel_hdcp_auth_downstream(struct intel_digital_port
>>*intel_dig_port,
>>
>> if (i == tries) {
>> DRM_ERROR("V Prime validation failed.(%d)\n", ret);
>>- return ret;
>>+ goto err;
>> }
>>
>> DRM_DEBUG_KMS("HDCP is enabled (%d downstream devices)\n",
>> num_downstream);
>>- return 0;
>>+ ret = 0;
>>+err:
>>+ kfree(ksv_fifo);
>>+ return ret;
>> }
>>
>> /* Implements Part 1 of the HDCP authorization procedure */
>>--
>>2.9.3
>
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
