diff options
| author | Alan Cox <alan@redhat.com> | 2008-10-13 10:40:43 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-13 09:51:41 -0700 |
| commit | 934e6ebf96e8c1a0f299e64129fdaebc1132a427 (patch) | |
| tree | ab4bd754997b097f06a5cfefd9e3671d56e628f4 /security | |
| parent | 2cb5998b5f0ccc886fdda3509059eef297b49577 (diff) | |
| download | lwn-934e6ebf96e8c1a0f299e64129fdaebc1132a427.tar.gz lwn-934e6ebf96e8c1a0f299e64129fdaebc1132a427.zip | |
tty: Redo current tty locking
Currently it is sometimes locked by the tty mutex and sometimes by the
sighand lock. The latter is in fact correct and now we can hand back referenced
objects we can fix this up without problems around sleeping functions.
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'security')
| -rw-r--r-- | security/selinux/hooks.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 089d61a23952..48881394fbd4 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c @@ -2121,9 +2121,7 @@ static inline void flush_unauthorized_files(struct files_struct *files) long j = -1; int drop_tty = 0; - mutex_lock(&tty_mutex); tty = get_current_tty(); - mutex_unlock(&tty_mutex); if (tty) { file_list_lock(); file = list_entry(tty->tty_files.next, typeof(*file), f_u.fu_list); |