diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-05-06 12:40:51 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-05-06 12:49:20 -0400 |
| commit | 00c81a58c5b4e0de14ee33bfbc3d71c90f69f9ea (patch) | |
| tree | d34d323e0a3b67ee5ce4c0ae8a0b3dbbc9bf2562 /kernel/trace/ring_buffer_benchmark.c | |
| parent | 8e7abf1c62941ebb7a1416cbc62392c8a0902625 (diff) | |
| download | op-kernel-dev-00c81a58c5b4e0de14ee33bfbc3d71c90f69f9ea.zip op-kernel-dev-00c81a58c5b4e0de14ee33bfbc3d71c90f69f9ea.tar.gz | |
ring-buffer: check for failed allocation in ring buffer benchmark
The result of the allocation of the ring buffer read page in the
ring buffer bench mark does not check the return to see if a page
was actually allocated. This patch fixes that.
[ Impact: avoid NULL dereference ]
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/ring_buffer_benchmark.c')
| -rw-r--r-- | kernel/trace/ring_buffer_benchmark.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/kernel/trace/ring_buffer_benchmark.c b/kernel/trace/ring_buffer_benchmark.c index 747244a..dcd75e9 100644 --- a/kernel/trace/ring_buffer_benchmark.c +++ b/kernel/trace/ring_buffer_benchmark.c @@ -84,6 +84,9 @@ static enum event_status read_page(int cpu) int i; bpage = ring_buffer_alloc_read_page(buffer); + if (!bpage) + return EVENT_DROPPED; + ret = ring_buffer_read_page(buffer, &bpage, PAGE_SIZE, cpu, 1); if (ret >= 0) { rpage = bpage; |
