[PATCH] EDAC: fix dev_set_name() format string
From: Arnd Bergmann
Date: Tue Mar 04 2025 - 09:45:32 EST
From: Arnd Bergmann <arnd@xxxxxxxx>
Passing a variable string as the format to dev_set_name() causes a W=1 warning:
drivers/edac/edac_device.c:736:9: error: format not a string literal and no format arguments [-Werror=format-security]
736 | ret = dev_set_name(&ctx->dev, name);
| ^~~
Use a literal "%s" instead so the name can be the argument.
Fixes: db99ea5f2c03 ("EDAC: Add support for EDAC device features control")
Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
---
drivers/edac/edac_device.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/edac/edac_device.c b/drivers/edac/edac_device.c
index 16611515ab34..0734909b08a4 100644
--- a/drivers/edac/edac_device.c
+++ b/drivers/edac/edac_device.c
@@ -733,7 +733,7 @@ int edac_dev_register(struct device *parent, char *name,
ctx->private = private;
dev_set_drvdata(&ctx->dev, ctx);
- ret = dev_set_name(&ctx->dev, name);
+ ret = dev_set_name(&ctx->dev, "%s", name);
if (ret)
goto data_mem_free;
--
2.39.5