>File *fp, opened but not closed, this patch add fclose(fp)
>
>Fixes: ef4b04f87fa6 ("crypto/ccp: support device init")
>Cc: sta...@dpdk.org
>
>Signed-off-by: Ferruh Yigit <ferruh.yi...@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

Reply via email to