diff options
| author | Haowen Bai <baihaowen@meizu.com> | 2022-04-21 16:21:17 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-04-22 16:42:30 +0200 |
| commit | 97042d0a75a1220bc2bb8f3cc99f3f72b1f81607 (patch) | |
| tree | 68adf2969f8cbd3a7624f94694dd43dfa4fb0de6 /tools/perf/scripts/python/exported-sql-viewer.py | |
| parent | staging: r8188eu: Drop redundant memset (diff) | |
| download | linux-97042d0a75a1220bc2bb8f3cc99f3f72b1f81607.tar.gz linux-97042d0a75a1220bc2bb8f3cc99f3f72b1f81607.zip | |
staging: rtl8192e: Fix signedness bug in rtllib_rx_assoc_resp()
The rtllib_rx_assoc_resp() function has a signedness bug because it's
a declared as a u16 but it return -ENOMEM. When you look at it more
closely it returns a mix of error codes including 0xcafe, -ENOMEM, and
a->status which is WLAN_STATUS_NOT_SUPPORTED_AUTH_ALG. This is a mess.
Clean it up to just return standard kernel error codes. We can print
out the a->status before returning a regular error code. The printks
in the caller need to be adjusted as well.
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Haowen Bai <baihaowen@meizu.com>
Link: https://lore.kernel.org/r/1650529277-7893-1-git-send-email-baihaowen@meizu.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions