[dpdk-dev] [PATCH] crypto/ccp: fix resource leak
Akhil Goyal
akhil.goyal at nxp.com
Mon Nov 12 13:49:23 CET 2018
On 11/4/2018 10:20 AM, Kumar, Ravi1 wrote:
>> File *fp, opened but not closed, this patch add fclose(fp)
>>
>> Fixes: ef4b04f87fa6 ("crypto/ccp: support device init")
>> Cc: stable at dpdk.org
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
>> ---
>> drivers/crypto/ccp/ccp_pci.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/crypto/ccp/ccp_pci.c b/drivers/crypto/ccp/ccp_pci.c index 59152ca5d..1702a09c4 100644
>> --- a/drivers/crypto/ccp/ccp_pci.c
>> +++ b/drivers/crypto/ccp/ccp_pci.c
>> @@ -31,12 +31,15 @@ ccp_check_pci_uio_module(void)
>> while (uio_module_names[i] != NULL) {
>> while (fgets(buf, sizeof(buf), fp) != NULL) {
>> if (!strncmp(buf, uio_module_names[i],
>> - strlen(uio_module_names[i])))
>> + strlen(uio_module_names[i]))) {
>> + fclose(fp);
>> return i;
>> + }
>> }
>> i++;
>> rewind(fp);
>> }
>> + fclose(fp);
>> printf("Insert igb_uio or uio_pci_generic kernel module(s)");
>> return -1;/* uio not inserted */
>> }
>> --
>> 2.17.2
>>
> Thanks Ferruh for the find. Looks good to me.
>
> Regards,
> Ravi
Added Reviewed-by from Ravi.
Applied to dpdk-next-crypto
Thanks
More information about the dev
mailing list