[dpdk-dev] [PATCH v2] Fix `eventfd_link' module leakages and races
Thomas Monjalon
thomas.monjalon at 6wind.com
Mon Mar 23 12:15:30 CET 2015
Hi Pavel,
You forgot the Signed-off-by line.
Huawei, Changchun, any comment on this patch?
2015-03-18 15:16, Pavel Boldin:
> The `eventfd_link' module provides an API to "steal" fd from another
> process had been written with a bug that leaks `struct file' because
> of the extra reference counter increment and missing `fput' call.
>
> The other bug is using another process' `task_struct' without incrementing
> a reference counter.
As there are 2 bugs, you should provide 2 patches.
> Fix these bugs and refactor the module.
Why refactoring along with the bug fixes?
You'd have more chances to have your fixes integrated in 2.0 without
refactoring.
Thanks
More information about the dev
mailing list