[LU-16915] sanity-sec test_51: Error: 'chown f51.sanity-sec' Created: 19/Jun/23  Updated: 30/Jan/24

Status: Open
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Minor
Reporter: Maloo Assignee: WC Triage
Resolution: Unresolved Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

This issue was created by maloo for Minh Diep <mdiep@whamcloud.com>

This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/c4e2f562-fc1a-4a0c-bb22-dd8f4261982b

test_51 failed with the following error:

chown f51.sanity-sec

Test session details:
clients: https://build.whamcloud.com/job/lustre-master/4435 - 5.14.0-284.11.1.el9_2.x86_64
servers: https://build.whamcloud.com/job/lustre-master/4435 - 5.14.0-284.11.1_lustre.el9.x86_64

<<Please provide additional information about the failure here>>

VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV
sanity-sec test_51 - chown f51.sanity-sec



 Comments   
Comment by Vladimir Saveliev [ 08/Dec/23 ]
== sanity-sec test 51: FS capabilities =================== 17:07:54 (1686330474)
running as uid/gid/euid/egid 500/500/500/500, groups:
 [/mnt/lustre/d51.sanity-sec/chown] [500] [/mnt/lustre/d51.sanity-sec/f51.sanity-sec]
/mnt/lustre/d51.sanity-sec/chown: changing ownership of '/mnt/lustre/d51.sanity-sec/f51.sanity-sec': Operation not permitted
running as uid/gid/euid/egid 500/500/500/500, groups:
 [/mnt/lustre/d51.sanity-sec/chown] [500] [/mnt/lustre/d51.sanity-sec/f51.sanity-sec]
execvp fails running /mnt/lustre/d51.sanity-sec/chown (22): Invalid argument
 sanity-sec test_51: @@@@@@ FAIL: chown f51.sanity-sec 

The “Invalid argument” comes from:

mdt_getxattr
  mdt_getxattr_all
    mo_xattr_get
      mdd_xattr_get
        mdo_xattr_get
          dt_xattr_get
            lod_xattr_get
              dt_xattr_get
                osd_xattr_get
                  __osd_xattr_get
                    ll_vfs_getxattr
                      vfs_getxattr
                        xattr_getsecurity
                          security_inode_getsecurity
                            cap_inode_getsecurity
                              dentry = d_find_any_alias(inode);
                              if (!dentry)
                                return -EINVAL;

ldiskfs inode does not pass specific security xattr handling vfs_getxattr() uses because ldiskfs’ inode has empty alias list.

Comment by Gerrit Updater [ 08/Dec/23 ]

"Vladimir Saveliev <vladimir.saveliev@hpe.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53382
Subject: LU-16915 osd-ldiskfs: use __vfs_getxattr in __osd_xattr_get
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3b4cd92f54bae5c303ef46d0d11bf0ff438e32f4

Generated at Sat Feb 10 03:31:03 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.