On Thu, Sep 20, 2018 at 6:46 PM, zhong jiang <zhongji...@huawei.com> wrote: > debugfs_remove_recursive has taken the null pointer into account. > just remove the null check before debugfs_remove_recursive. > > Acked-by: Masami Hiramatsu <mhira...@kernel.org> > Signed-off-by: zhong jiang <zhongji...@huawei.com>
Acked-by: Kees Cook <keesc...@chromium.org> -Kees > --- > kernel/fail_function.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/kernel/fail_function.c b/kernel/fail_function.c > index bc80a4e..17f75b5 100644 > --- a/kernel/fail_function.c > +++ b/kernel/fail_function.c > @@ -173,8 +173,7 @@ static void fei_debugfs_remove_attr(struct fei_attr *attr) > struct dentry *dir; > > dir = debugfs_lookup(attr->kp.symbol_name, fei_debugfs_dir); > - if (dir) > - debugfs_remove_recursive(dir); > + debugfs_remove_recursive(dir); > } > > static int fei_kprobe_handler(struct kprobe *kp, struct pt_regs *regs) > -- > 1.7.12.4 > -- Kees Cook Pixel Security