diff options
author | Eric Paris <eparis@redhat.com> | 2010-07-28 10:18:37 -0400 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2010-07-28 10:18:51 -0400 |
commit | 3bcf3860a4ff9bbc522820b4b765e65e4deceb3e (patch) | |
tree | 1e235af133559062c6fdee840ff9698f1dee26a6 /kernel/audit_watch.c | |
parent | f70ab54cc6c3907b0727ba332b3976f80f3846d0 (diff) |
fsnotify: store struct file not struct path
Al explains that calling dentry_open() with a mnt/dentry pair is only
garunteed to be safe if they are already used in an open struct file. To
make sure this is the case don't store and use a struct path in fsnotify,
always use a struct file.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'kernel/audit_watch.c')
-rw-r--r-- | kernel/audit_watch.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/audit_watch.c b/kernel/audit_watch.c index 7499397a6100..b955a22d8ff1 100644 --- a/kernel/audit_watch.c +++ b/kernel/audit_watch.c | |||
@@ -545,8 +545,8 @@ static int audit_watch_handle_event(struct fsnotify_group *group, struct fsnotif | |||
545 | return 0; | 545 | return 0; |
546 | 546 | ||
547 | switch (event->data_type) { | 547 | switch (event->data_type) { |
548 | case (FSNOTIFY_EVENT_PATH): | 548 | case (FSNOTIFY_EVENT_FILE): |
549 | inode = event->path.dentry->d_inode; | 549 | inode = event->file->f_path.dentry->d_inode; |
550 | break; | 550 | break; |
551 | case (FSNOTIFY_EVENT_INODE): | 551 | case (FSNOTIFY_EVENT_INODE): |
552 | inode = event->inode; | 552 | inode = event->inode; |