]> git.karo-electronics.de Git - karo-tx-linux.git/commit
inotify: fix race when adding a new watch
authorLino Sanfilippo <LinoSanfilippo@gmx.de>
Wed, 19 Jun 2013 00:05:30 +0000 (10:05 +1000)
committerStephen Rothwell <sfr@canb.auug.org.au>
Wed, 19 Jun 2013 07:12:43 +0000 (17:12 +1000)
commit70e34ab2dedc4e8b12bb733f8a4d7998e688377e
treec0181813634afe90e0d2bf1e76b78a1104ec7881
parent2541aedb503a0c7b5700a3e799fc7bda6f0242fc
inotify: fix race when adding a new watch

In inotify_new_watch() the number of watches for a group is compared
against the max number of allowed watches and increased afterwards.  The
check and incrementation is not done atomically, so it is possible for
multiple concurrent threads to pass the check and increment the number of
marks above the allowed max.

This patch uses an inotify groups mark_lock to ensure that both check and
incrementation are done atomic.  Furthermore we dont have to worry about
the race that allows a concurrent thread to add a watch just after
inotify_update_existing_watch() returned with -ENOENT anymore, since this
is also synchronized by the groups mark mutex now.

Signed-off-by: Lino Sanfilippo <LinoSanfilippo@gmx.de>
Cc: Eric Paris <eparis@redhat.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
fs/notify/inotify/inotify_user.c