Re: [PATCH] sys_execve and sys_uselib do not call into fsnotify

From: KOSAKI Motohiro
Date: Wed Dec 17 2008 - 21:39:42 EST


> On Wed, 17 Dec 2008 13:53:20 -0500
> Eric Paris <eparis@xxxxxxxxxx> wrote:
>
> > sys_execve and sys_uselib do not call into fsnotify so inotify does not get
> > open events for these types of syscalls. This patch simply makes the
> > requisite fsnotify calls.
>
> These two functions are grovelling around in pretty low-level fs
> operations. One wonders whether they could be converted (ie: cleaned
> up) to use do_sys_open() or some other such higher-level thing.
>
> That way, this fsnotify bug would be magically fixed, too.

hmm..

In current implementaion, do_sys_open() call fd_install().
then, it cause /proc/{pid}/fd entry change. iow, compatibility break.

Yup, it is fixable. but I think Eric's implementation is simpler.



--
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/