Skip to content
Snippets Groups Projects
Commit 00792f31 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman
Browse files

rtw88: Fix an error code in rtw_debugfs_set_rsvd_page()


[ Upstream commit c9eaee0c ]

The sscanf() function returns the number of matches (0 or 1 in this
case).  It doesn't return error codes.  We should return -EINVAL if the
string is invalid

Fixes: c376c1fc ("rtw88: add h2c command in debugfs")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/YE8nmatMDBDDWkjq@mwanda


Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 80cff3e1
No related branches found
No related tags found
No related merge requests found
...@@ -270,7 +270,7 @@ static ssize_t rtw_debugfs_set_rsvd_page(struct file *filp, ...@@ -270,7 +270,7 @@ static ssize_t rtw_debugfs_set_rsvd_page(struct file *filp,
if (num != 2) { if (num != 2) {
rtw_warn(rtwdev, "invalid arguments\n"); rtw_warn(rtwdev, "invalid arguments\n");
return num; return -EINVAL;
} }
debugfs_priv->rsvd_page.page_offset = offset; debugfs_priv->rsvd_page.page_offset = offset;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment