[PATCH] file: Add locking to f_getown
authorEric W. Biederman <ebiederm@xmission.com>
Mon, 2 Oct 2006 09:17:27 +0000 (02:17 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 2 Oct 2006 14:57:15 +0000 (07:57 -0700)
commit43fa1adb9334bf4585cd53144eb5911488f85bc7
tree64be64424e7db15e1872077f900870895ccac85b
parenta03fcb730b4fe7da14ca4405f23dbde717b1d2b9
[PATCH] file: Add locking to f_getown

This has been needed for a long time, but now with the advent of a
reference counted struct pid there are real consequences for getting this
wrong.

Someone I think it was Oleg Nesterov pointed out that this construct was
missing locking, when I introduced struct pid.  After taking time to review
the locking construct already present I figured out which lock needs to be
taken.  The other paths that access f_owner.pid take either the f_owner
read or the write lock.

Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Cc: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
fs/fcntl.c