diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-06-14 12:10:56 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-06-14 12:43:54 +0200 |
commit | ed6456afef0da57edfd87ce9142813d08a571137 (patch) | |
tree | 6e72930bc8f025aaeb6062028601a45d19b4de2d | |
parent | 64c667fe29ea697af0fce94857f26ae857951843 (diff) | |
download | linux-ed6456afef0da57edfd87ce9142813d08a571137.tar.gz linux-ed6456afef0da57edfd87ce9142813d08a571137.tar.bz2 linux-ed6456afef0da57edfd87ce9142813d08a571137.zip |
Staging: rtl8723bs: fix an error code in isFileReadable()
The caller only cares about zero vs non-zero so this code actually works
fine but we should be returning a negative error code instead of a valid
pointer casted to int.
Fixes: 554c0a3abf21 ("staging: Add rtl8723bs sdio wifi driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/rtl8723bs/os_dep/osdep_service.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/rtl8723bs/os_dep/osdep_service.c b/drivers/staging/rtl8723bs/os_dep/osdep_service.c index 02db59e8b593..aa16d1ab955b 100644 --- a/drivers/staging/rtl8723bs/os_dep/osdep_service.c +++ b/drivers/staging/rtl8723bs/os_dep/osdep_service.c @@ -160,7 +160,7 @@ static int isFileReadable(char *path) oldfs = get_fs(); set_fs(get_ds()); if (1!=readFile(fp, &buf, 1)) - ret = PTR_ERR(fp); + ret = -EINVAL; set_fs(oldfs); filp_close(fp, NULL); |