[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260107042458.1987818-2-den@valinux.co.jp>
Date: Wed, 7 Jan 2026 13:24:57 +0900
From: Koichiro Den <den@...inux.co.jp>
To: ntb@...ts.linux.dev
Cc: jdmason@...zu.us,
dave.jiang@...el.com,
allenbh@...il.com,
Frank.Li@....com,
linux-kernel@...r.kernel.org,
stable@...r.kernel.org
Subject: [PATCH v2 1/2] NTB: ntb_transport: Fix too small buffer for debugfs_name
The buffer used for "qp%d" was only 4 bytes, which truncates names like
"qp10" to "qp1" and causes multiple queues to share the same directory.
Enlarge the buffer and use sizeof() to avoid truncation.
Fixes: fce8a7bb5b4b ("PCI-Express Non-Transparent Bridge Support")
Cc: <stable@...r.kernel.org> # v3.9+
Reviewed-by: Frank Li <Frank.Li@....com>
Reviewed-by: Dave Jiang <dave.jiang@...el.com>
Signed-off-by: Koichiro Den <den@...inux.co.jp>
---
drivers/ntb/ntb_transport.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c
index eb875e3db2e3..857c845bfbe8 100644
--- a/drivers/ntb/ntb_transport.c
+++ b/drivers/ntb/ntb_transport.c
@@ -1236,9 +1236,9 @@ static int ntb_transport_init_queue(struct ntb_transport_ctx *nt,
qp->tx_max_entry = tx_size / qp->tx_max_frame;
if (nt->debugfs_node_dir) {
- char debugfs_name[4];
+ char debugfs_name[8];
- snprintf(debugfs_name, 4, "qp%d", qp_num);
+ snprintf(debugfs_name, sizeof(debugfs_name), "qp%d", qp_num);
qp->debugfs_dir = debugfs_create_dir(debugfs_name,
nt->debugfs_node_dir);
--
2.51.0
Powered by blists - more mailing lists