From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
Add the missing unlock before return from function simple_xattr_list()
in the error handling case.
Fixes: 786534b92f3c (tmpfs: listxattr should include POSIX ACL xattrs)
Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx>
Reviewed-by: Andreas Gruenbacher <agruenba@xxxxxxxxxx>
Cc: <stable@xxxxxxxxxxxxxxx> # 4.4
---
fs/xattr.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/fs/xattr.c b/fs/xattr.c
index 07d0e47..d4f8487 100644
--- a/fs/xattr.c
+++ b/fs/xattr.c
@@ -964,8 +964,10 @@ ssize_t simple_xattr_list(struct inode *inode, struct simple_xattrs *xattrs,
continue;
err = xattr_list_one(&buffer, &remaining_size, xattr->name);
- if (err)
+ if (err) {
+ spin_unlock(&xattrs->lock);
return err;
+ }
}
spin_unlock(&xattrs->lock);
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html