hurricos/openwrt-mirror
1
0
Fork 0
mirror of git://git.openwrt.org/openwrt/openwrt.git synced 2025-12-11 15:12:10 -05:00
Code Issues Projects Releases Wiki Activity

realtek: ethernet: Avoid usage of unsafe strcpy

Browse Source 
strcpy doesn't check the length of the destination buffer. And strlcpy
would not make sure to null-terminate the destination buffer.

Even when it is clear that this string will fit in the currrent buffer, it
is just best practice to avoid strcpy.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Link: https://github.com/openwrt/openwrt/pull/20906
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
This commit is contained in:
Sven Eckelmann 2025-11-23 15:10:41 +01:00 committed by Hauke Mehrtens
parent bc71e705aa
commit 80375e5acd
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c
View File

@ -1798,7 +1798,8 @@ static int __init rtl838x_eth_probe(struct platform_device *pdev)
} }
pr_info("Using MAC %08x%08x\n", sw_r32(priv->r->mac), pr_info("Using MAC %08x%08x\n", sw_r32(priv->r->mac),
sw_r32(priv->r->mac + 4)); sw_r32(priv->r->mac + 4));
strcpy(dev->name, "eth%d"); strscpy(dev->name, "eth%d", sizeof(dev->name));
priv->pdev = pdev; priv->pdev = pdev;
priv->netdev = dev; priv->netdev = dev;