bus: mhi: replace snprintf with sysfs_emit
authorWan Jiabing <wanjiabing@vivo.com>
Tue, 26 Apr 2022 12:58:59 +0000 (20:58 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 9 May 2022 13:41:01 +0000 (15:41 +0200)
Fix following coccicheck warning:
./drivers/bus/mhi/host/init.c:89:8-16: WARNING: use scnprintf or sprintf

Use sysfs_emit and sysfs_emit_at instead of snprintf.

Reviewed-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Manivannan Sadhasivam <mani@kernel.org>
Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
Link: https://lore.kernel.org/r/20220426125902.681258-1-wanjiabing@vivo.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/bus/mhi/host/init.c

index a665b8e..d2f1086 100644 (file)
@@ -86,7 +86,7 @@ static ssize_t serial_number_show(struct device *dev,
        struct mhi_device *mhi_dev = to_mhi_device(dev);
        struct mhi_controller *mhi_cntrl = mhi_dev->mhi_cntrl;
 
-       return snprintf(buf, PAGE_SIZE, "Serial Number: %u\n",
+       return sysfs_emit(buf, "Serial Number: %u\n",
                        mhi_cntrl->serial_number);
 }
 static DEVICE_ATTR_RO(serial_number);
@@ -100,9 +100,8 @@ static ssize_t oem_pk_hash_show(struct device *dev,
        int i, cnt = 0;
 
        for (i = 0; i < ARRAY_SIZE(mhi_cntrl->oem_pk_hash); i++)
-               cnt += snprintf(buf + cnt, PAGE_SIZE - cnt,
-                               "OEMPKHASH[%d]: 0x%x\n", i,
-                               mhi_cntrl->oem_pk_hash[i]);
+               cnt += sysfs_emit_at(buf, cnt, "OEMPKHASH[%d]: 0x%x\n",
+                               i, mhi_cntrl->oem_pk_hash[i]);
 
        return cnt;
 }