[tip:perf/urgent] perf: Fix off by one in perf_swevent_init()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: tip-bot for Dan Carpenter
Date: Thursday, December 16, 2010 - 3:42 am

Commit-ID:  ce677831a4abd0f9f957c90ac6f6a0d0472bafb4
Gitweb:     http://git.kernel.org/tip/ce677831a4abd0f9f957c90ac6f6a0d0472bafb4
Author:     Dan Carpenter <error27@gmail.com>
AuthorDate: Sun, 24 Oct 2010 21:50:42 +0200
Committer:  Ingo Molnar <mingo@elte.hu>
CommitDate: Thu, 16 Dec 2010 11:14:31 +0100

perf: Fix off by one in perf_swevent_init()

The perf_swevent_enabled[] array has PERF_COUNT_SW_MAX elements.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <20101024195041.GT5985@bicker>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 kernel/perf_event.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/kernel/perf_event.c b/kernel/perf_event.c
index 7b87017..2870fee 100644
--- a/kernel/perf_event.c
+++ b/kernel/perf_event.c
@@ -4719,7 +4719,7 @@ static int perf_swevent_init(struct perf_event *event)
 		break;
 	}
 
-	if (event_id > PERF_COUNT_SW_MAX)
+	if (event_id >= PERF_COUNT_SW_MAX)
 		return -ENOENT;
 
 	if (!event->parent) {
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch] perf: off by one in perf_swevent_init(), Dan Carpenter, (Sun Oct 24, 12:50 pm)
Re: [patch] perf: off by one in perf_swevent_init(), Dan Carpenter, (Mon Dec 13, 12:46 pm)
Re: [patch] perf: off by one in perf_swevent_init(), Peter Zijlstra, (Mon Dec 13, 3:38 pm)
[tip:perf/urgent] perf: Fix off by one in perf_swevent_init(), tip-bot for Dan Carp ..., (Thu Dec 16, 3:42 am)