diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2010-02-10 17:23:47 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-02-14 09:43:58 +0100 |
commit | a9bb18f36c8056f0712fb28c52c0f85d98438dfb (patch) | |
tree | 408900306e5730221cb026f9c8fe9bdc9ec14d7c | |
parent | tracing: Fix circular dead lock in stack trace (diff) | |
download | linux-a9bb18f36c8056f0712fb28c52c0f85d98438dfb.tar.xz linux-a9bb18f36c8056f0712fb28c52c0f85d98438dfb.zip |
tracing/kprobes: Fix probe parsing
Trying to add a probe like:
echo p:myprobe 0x10000 > /sys/kernel/debug/tracing/kprobe_events
will fail since the wrong pointer is passed to strict_strtoul
when trying to convert the address to an unsigned long.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Acked-by: Masami Hiramatsu <mhiramat@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
LKML-Reference: <20100210162346.GA6933@osiris.boeblingen.de.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/trace/trace_kprobe.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c index 6ea90c0e2c96..50b1b8239806 100644 --- a/kernel/trace/trace_kprobe.c +++ b/kernel/trace/trace_kprobe.c @@ -689,7 +689,7 @@ static int create_trace_probe(int argc, char **argv) return -EINVAL; } /* an address specified */ - ret = strict_strtoul(&argv[0][2], 0, (unsigned long *)&addr); + ret = strict_strtoul(&argv[1][0], 0, (unsigned long *)&addr); if (ret) { pr_info("Failed to parse address.\n"); return ret; |