raw/ioat: fix resource leak for directory handle
Checks
Commit Message
When reading the /dev directory as part of the bus scan for DSA devices,
the directory handle from opendir was not freed on function return,
leading to a resource leak.
Coverity issue: 370588
Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning automatically")
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
drivers/raw/ioat/idxd_bus.c | 1 +
1 file changed, 1 insertion(+)
Comments
06/05/2021 13:19, Bruce Richardson:
> When reading the /dev directory as part of the bus scan for DSA devices,
> the directory handle from opendir was not freed on function return,
> leading to a resource leak.
>
> Coverity issue: 370588
> Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning automatically")
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Applied, thanks
@@ -312,6 +312,7 @@ dsa_scan(void)
dev->device.numa_node = numa_node;
}
+ closedir(dev_dir);
return 0;
}