Messages in this thread Patch in this message | | | From | Qing Wang <> | Subject | [PATCH] media: ccs: replace snprintf in show functions with sysfs_emit | Date | Thu, 14 Oct 2021 23:49:22 -0700 |
| |
show() must not use snprintf() when formatting the value to be returned to user space.
Fix the following coccicheck warning: drivers/media/i2c/ccs/ccs-core.c:2717: WARNING: use scnprintf or sprintf.
Use sysfs_emit instead of scnprintf or sprintf makes more sense.
Signed-off-by: Qing Wang <wangqing@vivo.com> --- drivers/media/i2c/ccs/ccs-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c index a9403a2..21a156d 100644 --- a/drivers/media/i2c/ccs/ccs-core.c +++ b/drivers/media/i2c/ccs/ccs-core.c @@ -2715,11 +2715,11 @@ ccs_sysfs_ident_read(struct device *dev, struct device_attribute *attr, struct ccs_module_info *minfo = &sensor->minfo; if (minfo->mipi_manufacturer_id) - return snprintf(buf, PAGE_SIZE, "%4.4x%4.4x%2.2x\n", + return sysfs_emit(buf, "%4.4x%4.4x%2.2x\n", minfo->mipi_manufacturer_id, minfo->model_id, minfo->revision_number) + 1; else - return snprintf(buf, PAGE_SIZE, "%2.2x%4.4x%2.2x\n", + return sysfs_emit(buf, "%2.2x%4.4x%2.2x\n", minfo->smia_manufacturer_id, minfo->model_id, minfo->revision_number) + 1; } -- 2.7.4
| |