diff options
-rw-r--r-- | drivers/oprofile/event_buffer.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/drivers/oprofile/event_buffer.c b/drivers/oprofile/event_buffer.c index 2b7ae366ceb1..c38adb389064 100644 --- a/drivers/oprofile/event_buffer.c +++ b/drivers/oprofile/event_buffer.c | |||
@@ -41,6 +41,12 @@ static atomic_t buffer_ready = ATOMIC_INIT(0); | |||
41 | */ | 41 | */ |
42 | void add_event_entry(unsigned long value) | 42 | void add_event_entry(unsigned long value) |
43 | { | 43 | { |
44 | /* | ||
45 | * catch potential error | ||
46 | */ | ||
47 | if (!event_buffer) | ||
48 | return; | ||
49 | |||
44 | if (buffer_pos == buffer_size) { | 50 | if (buffer_pos == buffer_size) { |
45 | atomic_inc(&oprofile_stats.event_lost_overflow); | 51 | atomic_inc(&oprofile_stats.event_lost_overflow); |
46 | return; | 52 | return; |
@@ -92,9 +98,10 @@ out: | |||
92 | 98 | ||
93 | void free_event_buffer(void) | 99 | void free_event_buffer(void) |
94 | { | 100 | { |
101 | mutex_lock(&buffer_mutex); | ||
95 | vfree(event_buffer); | 102 | vfree(event_buffer); |
96 | |||
97 | event_buffer = NULL; | 103 | event_buffer = NULL; |
104 | mutex_unlock(&buffer_mutex); | ||
98 | } | 105 | } |
99 | 106 | ||
100 | 107 | ||
@@ -167,6 +174,11 @@ static ssize_t event_buffer_read(struct file *file, char __user *buf, | |||
167 | 174 | ||
168 | mutex_lock(&buffer_mutex); | 175 | mutex_lock(&buffer_mutex); |
169 | 176 | ||
177 | if (!event_buffer) { | ||
178 | retval = -EINTR; | ||
179 | goto out; | ||
180 | } | ||
181 | |||
170 | atomic_set(&buffer_ready, 0); | 182 | atomic_set(&buffer_ready, 0); |
171 | 183 | ||
172 | retval = -EFAULT; | 184 | retval = -EFAULT; |