[PATCH 1/3] coresight: Replace sprintf with scnprintf
Greg KH
gregkh at linuxfoundation.org
Sat Apr 8 11:42:23 EDT 2017
On Mon, Mar 27, 2017 at 11:09:31AM -0600, Mathieu Poirier wrote:
> From: Olav Haugan <ohaugan at codeaurora.org>
>
> Replace all occurrences of sprintf with scnprintf in coresight driver for
> consistency. scnprintf is also a safer alternative to sprintf.
But it's totally pointless here :(
>
> Signed-off-by: Olav Haugan <ohaugan at codeaurora.org>
> Signed-off-by: Mathieu Poirier <mathieu.poirier at linaro.org>
> ---
> drivers/hwtracing/coresight/coresight-etb10.c | 2 +-
> drivers/hwtracing/coresight/coresight-etm-perf.c | 2 +-
> .../hwtracing/coresight/coresight-etm3x-sysfs.c | 66 +++++++++++-----------
> drivers/hwtracing/coresight/coresight-funnel.c | 4 +-
> drivers/hwtracing/coresight/coresight-stm.c | 2 +-
> drivers/hwtracing/coresight/coresight-tmc.c | 2 +-
> 6 files changed, 39 insertions(+), 39 deletions(-)
>
> diff --git a/drivers/hwtracing/coresight/coresight-etb10.c b/drivers/hwtracing/coresight/coresight-etb10.c
> index d7325c6534ad..491f4da61c11 100644
> --- a/drivers/hwtracing/coresight/coresight-etb10.c
> +++ b/drivers/hwtracing/coresight/coresight-etb10.c
> @@ -588,7 +588,7 @@ static ssize_t trigger_cntr_show(struct device *dev,
> struct etb_drvdata *drvdata = dev_get_drvdata(dev->parent);
> unsigned long val = drvdata->trigger_cntr;
>
> - return sprintf(buf, "%#lx\n", val);
> + return scnprintf(buf, PAGE_SIZE, "%#lx\n", val);
There is no need for this in sysfs show callbacks. You _know_ this is a
PAGE_SIZE buffer, and that you can not overflow it with a simple number.
So don't change these types of functions for no reason please.
Same for most of this patch.
thanks,
greg k-h
More information about the linux-arm-kernel
mailing list