diff options
-rw-r--r-- | kernel/trace/trace.h | 4 | ||||
-rw-r--r-- | kernel/trace/trace_events.c | 5 | ||||
-rw-r--r-- | kernel/trace/trace_events_trigger.c | 7 |
3 files changed, 10 insertions, 6 deletions
diff --git a/kernel/trace/trace.h b/kernel/trace/trace.h index c56a773777ca..a6ef810e10e4 100644 --- a/kernel/trace/trace.h +++ b/kernel/trace/trace.h @@ -1023,6 +1023,10 @@ extern int event_trace_del_tracer(struct trace_array *tr); extern struct ftrace_event_file *find_event_file(struct trace_array *tr, const char *system, const char *event); +static inline void *event_file_data(struct file *filp) +{ + return ACCESS_ONCE(file_inode(filp)->i_private); +} extern struct mutex event_mutex; extern struct list_head ftrace_events; diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 6b4cb04882a9..a444f6588387 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -427,11 +427,6 @@ static void remove_subsystem(struct ftrace_subsystem_dir *dir) } } -static void *event_file_data(struct file *filp) -{ - return ACCESS_ONCE(file_inode(filp)->i_private); -} - static void remove_event_file_dir(struct ftrace_event_file *file) { struct dentry *dir = file->dir; diff --git a/kernel/trace/trace_events_trigger.c b/kernel/trace/trace_events_trigger.c index 54112be3376e..011bdfb5c76a 100644 --- a/kernel/trace/trace_events_trigger.c +++ b/kernel/trace/trace_events_trigger.c @@ -133,7 +133,12 @@ static int event_trigger_regex_open(struct inode *inode, struct file *file) mutex_lock(&event_mutex); - iter->file = inode->i_private; + iter->file = event_file_data(file); + if (!iter->file) { + kfree(iter); + mutex_unlock(&event_mutex); + return -ENODEV; + } if (file->f_mode & FMODE_READ) { ret = seq_open(file, &event_triggers_seq_ops); |