From: Dedy Lansky <[email protected]>

Replace sprintf with snprintf which checks the destination buffer
size.

Signed-off-by: Dedy Lansky <[email protected]>
Signed-off-by: Maya Erez <[email protected]>
---
 drivers/net/wireless/ath/wil6210/debugfs.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c 
b/drivers/net/wireless/ath/wil6210/debugfs.c
index 14778a1c..8eb6d49 100644
--- a/drivers/net/wireless/ath/wil6210/debugfs.c
+++ b/drivers/net/wireless/ath/wil6210/debugfs.c
@@ -976,9 +976,8 @@ static ssize_t wil_read_pmccfg(struct file *file, char 
__user *user_buf,
        " - \"alloc <num descriptors> <descriptor_size>\" to allocate pmc\n"
        " - \"free\" to free memory allocated for pmc\n";
 
-       sprintf(text, "Last command status: %d\n\n%s",
-               wil_pmc_last_cmd_status(wil),
-               help);
+       snprintf(text, sizeof(text), "Last command status: %d\n\n%s",
+                wil_pmc_last_cmd_status(wil), help);
 
        return simple_read_from_buffer(user_buf, count, ppos, text,
                                       strlen(text) + 1);
-- 
1.9.1

Reply via email to