From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761282AbYDRUJB (ORCPT ); Fri, 18 Apr 2008 16:09:01 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1760307AbYDRUIv (ORCPT ); Fri, 18 Apr 2008 16:08:51 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.122]:52269 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757710AbYDRUIv (ORCPT ); Fri, 18 Apr 2008 16:08:51 -0400 Message-Id: <20080418200849.968033091@goodmis.org> References: <20080418200538.788522018@goodmis.org> User-Agent: quilt/0.46-1 Date: Fri, 18 Apr 2008 16:05:39 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Ingo Molnar , Steven Rostedt , akpm@osdl.org, Peter Zijlstra , Soeren Sandmann Pedersen , Pekka Paalanen , Steven Rostedt Subject: [PATCH 1/5] ftrace: simple clean ups Content-Disposition: inline; filename=ftrace-small-clean-ups.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Andrew Morton mentioned some clean ups that should be done to ftrace. This patch does some of the simple clean ups. Signed-off-by: Steven Rostedt --- kernel/trace/trace.c | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) Index: linux-sched-devel.git/kernel/trace/trace.c =================================================================== --- linux-sched-devel.git.orig/kernel/trace/trace.c 2008-04-18 15:47:22.000000000 -0400 +++ linux-sched-devel.git/kernel/trace/trace.c 2008-04-18 15:47:40.000000000 -0400 @@ -36,8 +36,7 @@ unsigned long __read_mostly tracing_max_ unsigned long __read_mostly tracing_thresh; /* dummy trace to disable tracing */ -static struct tracer no_tracer __read_mostly = -{ +static struct tracer no_tracer __read_mostly = { .name = "none", }; @@ -1961,8 +1960,8 @@ tracing_iter_ctrl_write(struct file *fil int neg = 0; int i; - if (cnt > 63) - cnt = 63; + if (cnt >= sizeof(buf)) + return -EINVAL; if (copy_from_user(&buf, ubuf, cnt)) return -EFAULT; @@ -2054,8 +2053,8 @@ tracing_ctrl_write(struct file *filp, co long val; char buf[64]; - if (cnt > 63) - cnt = 63; + if (cnt >= sizeof(buf)) + return -EINVAL; if (copy_from_user(&buf, ubuf, cnt)) return -EFAULT; @@ -2154,10 +2153,10 @@ tracing_max_lat_read(struct file *filp, char buf[64]; int r; - r = snprintf(buf, 64, "%ld\n", + r = snprintf(buf, sizeof(buf), "%ld\n", *ptr == (unsigned long)-1 ? -1 : nsecs_to_usecs(*ptr)); - if (r > 64) - r = 64; + if (r > sizeof(buf)) + r = sizeof(buf); return simple_read_from_buffer(ubuf, cnt, ppos, buf, r); } @@ -2169,8 +2168,8 @@ tracing_max_lat_write(struct file *filp, long val; char buf[64]; - if (cnt > 63) - cnt = 63; + if (cnt >= sizeof(buf)) + return -EINVAL; if (copy_from_user(&buf, ubuf, cnt)) return -EFAULT; @@ -2434,8 +2433,8 @@ tracing_entries_write(struct file *filp, unsigned long val; char buf[64]; - if (cnt > 63) - cnt = 63; + if (cnt >= sizeof(buf)) + return -EINVAL; if (copy_from_user(&buf, ubuf, cnt)) return -EFAULT; --