[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20220524132947.03582de5@gandalf.local.home>
Date: Tue, 24 May 2022 13:29:47 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Yuntao Wang <ytcoode@...il.com>
Cc: Ingo Molnar <mingo@...hat.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] tracing: Use strim() to remove whitespace instead of
doing it manually
On Fri, 21 Jan 2022 09:56:23 +0000
Yuntao Wang <ytcoode@...il.com> wrote:
> The tracing_set_trace_write() function just removes the trailing whitespace
> from the user supplied tracer name, but the leading whitespace should also
> be removed.
>
> In addition, if the user supplied tracer name contains only a few
> whitespace characters, the first one will not be removed using the current
> method, which results it a single whitespace character left in the buf.
>
> To fix all of these issues, we use strim() to correctly remove both the
> leading and trailing whitespace.
>
> Signed-off-by: Yuntao Wang <ytcoode@...il.com>
Added to my queue.
Thanks,
-- Steve
Powered by blists - more mailing lists