2023-02-10 08:04:44

by Andrew Donnellan

[permalink] [raw]
Subject: [PATCH v6 06/26] powerpc/secvar: Use sysfs_emit() instead of sprintf()

From: Russell Currey <[email protected]>

The secvar format string and object size sysfs files are both ASCII
text, and should use sysfs_emit(). No functional change.

Suggested-by: Greg Kroah-Hartman <[email protected]>
Signed-off-by: Russell Currey <[email protected]>
Signed-off-by: Andrew Donnellan <[email protected]>
Reviewed-by: Stefan Berger <[email protected]>

---

v2: New patch (gregkh)
---
arch/powerpc/kernel/secvar-sysfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/kernel/secvar-sysfs.c b/arch/powerpc/kernel/secvar-sysfs.c
index 702044edf14d..b786d1005027 100644
--- a/arch/powerpc/kernel/secvar-sysfs.c
+++ b/arch/powerpc/kernel/secvar-sysfs.c
@@ -35,7 +35,7 @@ static ssize_t format_show(struct kobject *kobj, struct kobj_attribute *attr,
if (rc)
goto out;

- rc = sprintf(buf, "%s\n", format);
+ rc = sysfs_emit(buf, "%s\n", format);

out:
of_node_put(node);
@@ -57,7 +57,7 @@ static ssize_t size_show(struct kobject *kobj, struct kobj_attribute *attr,
return rc;
}

- return sprintf(buf, "%llu\n", dsize);
+ return sysfs_emit(buf, "%llu\n", dsize);
}

static ssize_t data_read(struct file *filep, struct kobject *kobj,
--
2.39.1