Re: [PATCH] btrfs-progs: mkfs rootdir: use lgetxattr() not to follow a symbolic link
On Mon, Apr 02, 2018 at 10:59:31AM +0900, Misono Tomohiro wrote: > mkfs-test 016 "rootdir-bad-symbolic-link" fails when selinux is enabled. > This is because add_xattr_item() uses getxattr() and tries to follow a > bad symbolic link for selinux item, which causes ENOENT error. > > The line above already uses llistxattr() for getting list of xattr in > order not to follow a symbolic link, so just use lgetxattr() too. > > Signed-off-by: Tomohiro MisonoApplied and added to 4.16, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] btrfs-progs: mkfs rootdir: use lgetxattr() not to follow a symbolic link
On 2018年04月02日 09:59, Misono Tomohiro wrote: > mkfs-test 016 "rootdir-bad-symbolic-link" fails when selinux is enabled. > This is because add_xattr_item() uses getxattr() and tries to follow a > bad symbolic link for selinux item, which causes ENOENT error. > > The line above already uses llistxattr() for getting list of xattr in > order not to follow a symbolic link, so just use lgetxattr() too. > > Signed-off-by: Tomohiro MisonoReviewed-by: Qu Wenruo Thanks, Qu > --- > mkfs/rootdir.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mkfs/rootdir.c b/mkfs/rootdir.c > index 33c3ff1e..ff00bb0f 100644 > --- a/mkfs/rootdir.c > +++ b/mkfs/rootdir.c > @@ -249,7 +249,7 @@ static int add_xattr_item(struct btrfs_trans_handle > *trans, > cur_name_len = strlen(cur_name); > next_location += cur_name_len + 1; > > - ret = getxattr(file_name, cur_name, cur_value, XATTR_SIZE_MAX); > + ret = lgetxattr(file_name, cur_name, cur_value, XATTR_SIZE_MAX); > if (ret < 0) { > if (errno == ENOTSUP) > return 0; > signature.asc Description: OpenPGP digital signature
[PATCH] btrfs-progs: mkfs rootdir: use lgetxattr() not to follow a symbolic link
mkfs-test 016 "rootdir-bad-symbolic-link" fails when selinux is enabled. This is because add_xattr_item() uses getxattr() and tries to follow a bad symbolic link for selinux item, which causes ENOENT error. The line above already uses llistxattr() for getting list of xattr in order not to follow a symbolic link, so just use lgetxattr() too. Signed-off-by: Tomohiro Misono--- mkfs/rootdir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkfs/rootdir.c b/mkfs/rootdir.c index 33c3ff1e..ff00bb0f 100644 --- a/mkfs/rootdir.c +++ b/mkfs/rootdir.c @@ -249,7 +249,7 @@ static int add_xattr_item(struct btrfs_trans_handle *trans, cur_name_len = strlen(cur_name); next_location += cur_name_len + 1; - ret = getxattr(file_name, cur_name, cur_value, XATTR_SIZE_MAX); + ret = lgetxattr(file_name, cur_name, cur_value, XATTR_SIZE_MAX); if (ret < 0) { if (errno == ENOTSUP) return 0; -- 2.14.3 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html