diff options
| author | Zhang Xiaoxu <zhangxiaoxu5@huawei.com> | 2022-11-15 18:39:34 +0800 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2022-11-16 00:21:04 -0600 |
| commit | 92bbd67a55fee50743b42825d1c016e7fd5c79f9 (patch) | |
| tree | 2f26e1e6bf386d89495a003d2b6b353355c5f707 /fs/cifs/ioctl.c | |
| parent | d520de6cb42e88a1d008b54f935caf9fc05951da (diff) | |
| download | linux-92bbd67a55fee50743b42825d1c016e7fd5c79f9.tar.bz2 | |
cifs: Fix wrong return value checking when GETFLAGS
The return value of CIFSGetExtAttr is negative, should be checked
with -EOPNOTSUPP rather than EOPNOTSUPP.
Fixes: 64a5cfa6db94 ("Allow setting per-file compression via SMB2/3")
Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5@huawei.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/ioctl.c')
| -rw-r--r-- | fs/cifs/ioctl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/cifs/ioctl.c b/fs/cifs/ioctl.c index 89d5fa887364..6419ec47c2a8 100644 --- a/fs/cifs/ioctl.c +++ b/fs/cifs/ioctl.c @@ -343,7 +343,7 @@ long cifs_ioctl(struct file *filep, unsigned int command, unsigned long arg) rc = put_user(ExtAttrBits & FS_FL_USER_VISIBLE, (int __user *)arg); - if (rc != EOPNOTSUPP) + if (rc != -EOPNOTSUPP) break; } #endif /* CONFIG_CIFS_ALLOW_INSECURE_LEGACY */ @@ -373,7 +373,7 @@ long cifs_ioctl(struct file *filep, unsigned int command, unsigned long arg) * pSMBFile->fid.netfid, * extAttrBits, * &ExtAttrMask); - * if (rc != EOPNOTSUPP) + * if (rc != -EOPNOTSUPP) * break; */ |