On Mon, Jun 8, 2026 at 4:55 AM [off-list ref] wrote:
quoted hunk ↗ jump to hunk
diff --git a/drivers/net/ethernet/qualcomm/emac/emac-ethtool.c b/drivers/net/ethernet/qualcomm/emac/emac-ethtool.c
index f502db9cdea9..ffc300641aac 100644
--- a/drivers/net/ethernet/qualcomm/emac/emac-ethtool.c
+++ b/drivers/net/ethernet/qualcomm/emac/emac-ethtool.c
@@ -95,7 +95,7 @@ static void emac_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
switch (stringset) {
case ETH_SS_PRIV_FLAGS:
- strcpy(data, "single-pause-mode");
+ strscpy(data, "single-pause-mode", ETH_GSTRING_LEN);
break;
It would be better to replace this with a call to ethtool_sprintf(),
which already takes ETH_GSTRING_LEN into account. And may as well
update the ETH_SS_STATS case underneath it.
case ETH_SS_STATS:
--
2.39.5