Re: [PATCH resend] audit: fix mark refcounting

[Miklos Szeredi]

Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes:

> Looks reasonable, but why doesn't all callers have that "put_mark()" thing?
>
> And if/when all callers *do* have that put_mark() thing, maybe we
> should make destroy_mark() just do it?
>
> In particular, a quick grep shows that there are destroy_mark users still in:
>
>  - fs/notify/fanotify/fanotify_user.c
>
>  - fs/notify/dnotify/dnotify.c (2 of them)

These do in fact do "put_mark()" after the "destroy_mark()".

>
>  - fs/notify/inotify/inotify_fsnotify.c

This one, I think, is broken.

Thanks,
Miklos
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/