aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Straube <[email protected]>2023-10-23 11:00:01 +0200
committerGreg Kroah-Hartman <[email protected]>2023-10-25 12:57:46 +0200
commite26511f672432417cc1286e00bc65ab82012f5af (patch)
tree0a2158ee7289dde3da02bb6f41a1057bb8398f43
parent5e3441ebe93c82f85e582eed8e34eb7eee541e7e (diff)
staging: vme_user: prefer strscpy over strcpy
Using strcpy has potential for buffer overflows. It should be replaced with strscpy where possible. In this case the return value of strcpy is not used, so we can safely replace it with strscpy. Signed-off-by: Michael Straube <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
-rw-r--r--drivers/staging/vme_user/vme_tsi148.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/vme_user/vme_tsi148.c b/drivers/staging/vme_user/vme_tsi148.c
index d549c22da534..2ec9c2904404 100644
--- a/drivers/staging/vme_user/vme_tsi148.c
+++ b/drivers/staging/vme_user/vme_tsi148.c
@@ -2310,7 +2310,7 @@ static int tsi148_probe(struct pci_dev *pdev, const struct pci_device_id *id)
mutex_init(&tsi148_device->vme_rmw);
tsi148_bridge->parent = &pdev->dev;
- strcpy(tsi148_bridge->name, driver_name);
+ strscpy(tsi148_bridge->name, driver_name, VMENAMSIZ);
/* Setup IRQ */
retval = tsi148_irq_init(tsi148_bridge);