Re: [PATCH] s390: replace snprintf in show functions with sysfs_emit

From: Heiko Carstens
Date: Mon Oct 18 2021 - 06:48:40 EST


On Thu, Oct 14, 2021 at 11:50:54PM -0700, Qing Wang wrote:
> show() must not use snprintf() when formatting the value to be
> returned to user space.
>
> Fix the coccicheck warnings:
> WARNING: use scnprintf or sprintf.
>
> Use sysfs_emit instead of scnprintf or sprintf makes more sense.
>
> Signed-off-by: Qing Wang <wangqing@xxxxxxxx>
> ---
> drivers/s390/block/dasd_devmap.c | 74 ++++++++++++++++++++--------------------
> drivers/s390/char/raw3270.c | 6 ++--
> drivers/s390/cio/chp.c | 2 +-
> 3 files changed, 41 insertions(+), 41 deletions(-)

This requires either Reviewed-by or Acked-by from dasd and cio
maintainers, which you already cc'ed.