cifs: Fix wrong return value checking when GETFLAGS
[ Upstream commit 92bbd67a55fee50743b42825d1c016e7fd5c79f9 ]
The return value of CIFSGetExtAttr is negative, should be checked
with -EOPNOTSUPP rather than EOPNOTSUPP.
Fixes: 64a5cfa6db
("Allow setting per-file compression via SMB2/3")
Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5@huawei.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
9f00da9c86
commit
e13ef43813
@@ -193,7 +193,7 @@ long cifs_ioctl(struct file *filep, unsigned int command, unsigned long arg)
|
|||||||
rc = put_user(ExtAttrBits &
|
rc = put_user(ExtAttrBits &
|
||||||
FS_FL_USER_VISIBLE,
|
FS_FL_USER_VISIBLE,
|
||||||
(int __user *)arg);
|
(int __user *)arg);
|
||||||
if (rc != EOPNOTSUPP)
|
if (rc != -EOPNOTSUPP)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
#endif /* CONFIG_CIFS_POSIX */
|
#endif /* CONFIG_CIFS_POSIX */
|
||||||
@@ -222,7 +222,7 @@ long cifs_ioctl(struct file *filep, unsigned int command, unsigned long arg)
|
|||||||
* pSMBFile->fid.netfid,
|
* pSMBFile->fid.netfid,
|
||||||
* extAttrBits,
|
* extAttrBits,
|
||||||
* &ExtAttrMask);
|
* &ExtAttrMask);
|
||||||
* if (rc != EOPNOTSUPP)
|
* if (rc != -EOPNOTSUPP)
|
||||||
* break;
|
* break;
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user