]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
platform/x86/intel/uncore-freq: Replace sprintf() with sysfs_emit()
authorKaushlendra Kumar <kaushlendra.kumar@intel.com>
Tue, 30 Dec 2025 12:45:16 +0000 (18:15 +0530)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Tue, 6 Jan 2026 08:57:11 +0000 (10:57 +0200)
Replace sprintf() with sysfs_emit() in sysfs show functions. The
sysfs_emit() function is the preferred way to format sysfs output as
it ensures proper buffer bounds checking and correct return values.

Signed-off-by: Kaushlendra Kumar <kaushlendra.kumar@intel.com>
Link: https://patch.msgid.link/20251230124516.229125-3-kaushlendra.kumar@intel.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/intel/uncore-frequency/uncore-frequency-common.c

index e9495ac5ecd058d1270144521d5a02573720a201..7070c94324e0edbf18bf093ffbaff455738b20b3 100644 (file)
@@ -26,21 +26,21 @@ static ssize_t show_domain_id(struct kobject *kobj, struct kobj_attribute *attr,
 {
        struct uncore_data *data = container_of(attr, struct uncore_data, domain_id_kobj_attr);
 
-       return sprintf(buf, "%u\n", data->domain_id);
+       return sysfs_emit(buf, "%u\n", data->domain_id);
 }
 
 static ssize_t show_fabric_cluster_id(struct kobject *kobj, struct kobj_attribute *attr, char *buf)
 {
        struct uncore_data *data = container_of(attr, struct uncore_data, fabric_cluster_id_kobj_attr);
 
-       return sprintf(buf, "%u\n", data->cluster_id);
+       return sysfs_emit(buf, "%u\n", data->cluster_id);
 }
 
 static ssize_t show_package_id(struct kobject *kobj, struct kobj_attribute *attr, char *buf)
 {
        struct uncore_data *data = container_of(attr, struct uncore_data, package_id_kobj_attr);
 
-       return sprintf(buf, "%u\n", data->package_id);
+       return sysfs_emit(buf, "%u\n", data->package_id);
 }
 
 #define MAX_UNCORE_AGENT_TYPES 4
@@ -77,7 +77,7 @@ static ssize_t show_attr(struct uncore_data *data, char *buf, enum uncore_index
        if (ret)
                return ret;
 
-       return sprintf(buf, "%u\n", value);
+       return sysfs_emit(buf, "%u\n", value);
 }
 
 static ssize_t store_attr(struct uncore_data *data, const char *buf, ssize_t count,