[PATCH REQUEST 4.7.y, 4.8.y] xfs: Make __xfs_xattr_put_listen preperly report errors.

DORMANTno replies

From: Eric Sandeen <hidden>
Date: 2016-10-18 21:31:43

This commit should apply back to v4.7; it fixes a regression introduced
in that release which causes serious problems with storing & retrieving
xattrs.  If it could be applied to any stable kernels for 4.7 and 4.8
that'd be great.

Thanks,
-Eric

commit 791cc43b36eb1f88166c8505900cad1b43c7fe1a
Author: Artem Savkov [off-list ref]
Date:   Wed Sep 14 07:40:35 2016 +1000

    Make __xfs_xattr_put_listen preperly report errors.
    
    Commit 2a6fba6 "xfs: only return -errno or success from attr ->put_listent"
    changes the returnvalue of __xfs_xattr_put_listen to 0 in case when there is
    insufficient space in the buffer assuming that setting context->count to -1
    would be enough, but all of the ->put_listent callers only check seen_enough.
    This results in a failed assertion:
    XFS: Assertion failed: context->count >= 0, file: fs/xfs/xfs_xattr.c, line: 175
    in insufficient buffer size case.
    
    This is only reproducible with at least 2 xattrs and only when the buffer
    gets depleted before the last one.
    
    Furthermore if buffersize is such that it is enough to hold the last xattr's
    name, but not enough to hold the sum of preceeding xattr names listxattr won't
    fail with ERANGE, but will suceed returning last xattr's name without the
    first character. The first character end's up overwriting data stored at
    (context->alist - 1).
    
    Signed-off-by: Artem Savkov [off-list ref]
    Reviewed-by: Dave Chinner [off-list ref]
    Signed-off-by: Dave Chinner [off-list ref]
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help