From 4f5bf3d1478dfcfd4d8023d15f5627adc11d5d9a Mon Sep 17 00:00:00 2001 From: Alexander Winkowski Date: Mon, 4 Mar 2024 17:51:12 +0000 Subject: [PATCH] coresight-tmc: Fix false positive -Wfortify-source The size is checked just above. ../drivers/hwtracing/coresight/coresight-tmc.c:423:26: warning: 'sscanf' may overflow; destination buffer in argument 3 has size 10, but the corresponding specifier may require size 11 [-Wfortify-source] 423 | if (sscanf(buf, "%10s", str) != 1) | ^ ../drivers/hwtracing/coresight/coresight-tmc.c:464:26: warning: 'sscanf' may overflow; destination buffer in argument 3 has size 10, but the corresponding specifier may require size 11 [-Wfortify-source] 464 | if (sscanf(buf, "%10s", str) != 1) | ^ 2 warnings generated. Change-Id: I5db199a85ba0c7dfc15fae5d62c9c4d8c550d3e7 Signed-off-by: Alexander Winkowski --- drivers/hwtracing/coresight/coresight-tmc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hwtracing/coresight/coresight-tmc.c b/drivers/hwtracing/coresight/coresight-tmc.c index 05a7cbfd63ec..8e0726c97c06 100644 --- a/drivers/hwtracing/coresight/coresight-tmc.c +++ b/drivers/hwtracing/coresight/coresight-tmc.c @@ -420,7 +420,7 @@ static ssize_t out_mode_store(struct device *dev, if (strlen(buf) >= 10) return -EINVAL; - if (sscanf(buf, "%10s", str) != 1) + if (sscanf(buf, "%s", str) != 1) return -EINVAL; ret = tmc_etr_switch_mode(drvdata, str); return ret ? ret : size; @@ -461,7 +461,7 @@ static ssize_t pcie_path_store(struct device *dev, if (strlen(buf) >= 10) return -EINVAL; - if (sscanf(buf, "%10s", str) != 1) + if (sscanf(buf, "%s", str) != 1) return -EINVAL; mutex_lock(&drvdata->mem_lock);