<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>13959</bug_id>
          
          <creation_ts>2020-06-26 02:02:13 +0000</creation_ts>
          <short_desc>ACLs: setfacl -m on a directory corrupts UID&amp;GID in pseudo</short_desc>
          <delta_ts>2020-06-28 00:40:41 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Pseudo</product>
          <component>pseudo</component>
          <version>master</version>
          <rep_platform>x86</rep_platform>
          <op_sys>x86_64</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>3.2 M2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Johannes Beisswenger">johannes.beisswenger</reporter>
          <assigned_to name="Richard Purdie">richard.purdie</assigned_to>
          <cc>johannes.beisswenger</cc>
    
    <cc>richard.purdie</cc>
    
    <cc>seebs</cc>
    
    <cc>yp.pseudo.watcher</cc>
    
    <cc>yp.watcher</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>New (Never tested)</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>87624</commentid>
    <comment_count>0</comment_count>
    <who name="Johannes Beisswenger">johannes.beisswenger</who>
    <bug_when>2020-06-26 02:02:13 +0000</bug_when>
    <thetext>Preconditions/Environment
-------------------------
Ubuntu 18.04.4 LTS (Bionic Beaver)
pseudo at commit 060058b
With 0001-pseudo_ipc.h-Fix-enum-typedef.patch from poky(http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=b4e1ce1b93b08c77047508f27915073db6315a8e) applied.

Triggering Action/Cause
-----------------------
Run `pseudo`.
Run `mkdir foobar`.
Run `setfacl -m &quot;u:root:rX&quot; foobar`.

Expectation
-----------
UID/GID of foobar should be 0/root.

`stat foobar`:
  File: foobar/
  Size: 4096      	Blocks: 8          IO Block: 4096   directory
...
Access: (0775/drwxrwxr-x)  Uid: (    0/    root)   Gid: (    0/    root)
...

`getfacl foobar`:
# file: foobar/
# owner: root
# group: root
user::rwx
user:root:r-x
group::rwx
mask::rwx
other::r-x

Actual Result
-------------
UID/GID of foobar is 1001/jbe, the user running pseudo.

`getfacl foobar`:
# file: foobar/
# owner: jbe
# group: jbe
user::rwx
user:root:r-x
group::rwx
mask::rwx
other::r-x

`stat foobar`:
  File: foobar/
  Size: 4096      	Blocks: 8          IO Block: 4096   directory
...
Access: (0755/drwxr-xr-x)  Uid: ( 1001/     jbe)   Gid: ( 1001/     jbe)
...


Reproducibility
---------------
This is 100% reproducible when restarting pseudo in between trials or when using a previously not used directory name.

Note: I have a patch fixing this issue which I will submit to the mailing list.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87625</commentid>
    <comment_count>1</comment_count>
      <attachid>4696</attachid>
    <who name="Johannes Beisswenger">johannes.beisswenger</who>
    <bug_when>2020-06-26 04:18:20 +0000</bug_when>
    <thetext>Created attachment 4696
Pseudo testcase that can be used to reproduce the issue

This is the testcase from this PATCH https://lists.openembedded.org/g/openembedded-core/message/140009.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87626</commentid>
    <comment_count>2</comment_count>
    <who name="Johannes Beisswenger">johannes.beisswenger</who>
    <bug_when>2020-06-26 04:22:07 +0000</bug_when>
    <thetext>Fix https://lists.openembedded.org/g/openembedded-core/message/140008</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>87648</commentid>
    <comment_count>3</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2020-06-28 00:40:41 +0000</bug_when>
    <thetext>Thanks, we&apos;ve merged this into s staging branch and applied to OE-Core.

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=704a245460f1802cb83a1a7200d1bbfb5bc0c33c</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>4696</attachid>
            <date>2020-06-26 04:18:20 +0000</date>
            <delta_ts>2020-06-26 04:18:20 +0000</delta_ts>
            <desc>Pseudo testcase that can be used to reproduce the issue</desc>
            <filename>test-acl.sh</filename>
            <type>application/x-shellscript</type>
            <size>4977</size>
            <attacher name="Johannes Beisswenger">johannes.beisswenger</attacher>
            
              <data encoding="base64">IyEvYmluL2Jhc2gKIwojIFNQRFgtTGljZW5zZS1JZGVudGlmaWVyOiBMR1BMLTIuMS1vbmx5CiMK
CiMgUmV0dXJuIHZhbHM6IDIgLSBVbmFibGUgdG8gcnVuIEFDTCBjb21tYW5kcywgYXNzZXJ0aW9u
IGZhaWx1cmUKIyAgICAgICAgICAgICAgMSAtIEludmFsaWQgcmV0dXJuIHZhbHVlCiMgICAgICAg
ICAgICAgIDAgLSBQYXNzCgojIE5PVEU6IHRoZXNlIHRlc3QgZXhjbHVzaXZlbHkgdGVzdCBzZXRm
YWNsIC1tCgpzZXQgLXUKCmNoZWNrX293bmVyICgpIHsKICAgIGxvY2FsIGZpbGU9IiQxIgogICAg
bG9jYWwgZXhwZWN0ZWQ9IiQyIgogICAgbG9jYWwgbXNnPSIkMyIKICAgIGxvY2FsIGFjdHVhbD0k
KHN0YXQgLWMgIiVVIiAiJGZpbGUiKQogICAgaWYgWyAiJGFjdHVhbCIgIT0gIiRleHBlY3RlZCIg
XQogICAgdGhlbgogICAgICAgIGVjaG8gIiRtc2ciICJGYWlsLCAnJGZpbGUnIHVuZXhwZWN0ZWQg
b3duZXIgJyRhY3R1YWwnIgogICAgICAgIGV4aXQgMgogICAgZmkKfQoKY2hlY2tfZ3JvdXAgKCkg
ewogICAgbG9jYWwgZmlsZT0iJDEiCiAgICBsb2NhbCBleHBlY3RlZD0iJDIiCiAgICBsb2NhbCBt
c2c9IiQzIgogICAgbG9jYWwgYWN0dWFsPSQoc3RhdCAtYyAiJUciICIkZmlsZSIpCiAgICBpZiBb
ICIkYWN0dWFsIiAhPSAiJGV4cGVjdGVkIiBdCiAgICB0aGVuCiAgICAgICAgZWNobyAiJG1zZyIg
IkZhaWwsICckZmlsZScgdW5leHBlY3RlZCBncm91cCAnJGFjdHVhbCciCiAgICAgICAgZXhpdCAy
CiAgICBmaQp9CgpjaGVja19hY2xfY29udGFpbnMgKCkgewogICAgbG9jYWwgZmlsZT0iJDEiCiAg
ICBsb2NhbCBhY2w9IiQyIgogICAgbG9jYWwgbXNnPSIkMyIKICAgIElGUz0nLCcgcmVhZCAtcmEg
YWNscyA8PDwgIiRhY2wiCiAgICBmb3IgcGF0dGVybiBpbiAiJHthY2xzW0BdfSI7IGRvCiAgICAg
ICAgcmVzdWx0PSQoZ2V0ZmFjbCAtYyAiJGZpbGUiIHwgZ3JlcCAtbyAiXiRwYXR0ZXJuIikKICAg
ICAgICBpZiBbICIkcmVzdWx0IiAhPSAiJHBhdHRlcm4iIF0KICAgICAgICB0aGVuCiAgICAgICAg
ICAgIGVjaG8gIiRtc2ciICJGYWlsLCBkaWQgbm90IGZpbmQgZGVzaXJlZCBhY2wgJyRwYXR0ZXJu
JyBpbiAnJGZpbGUnIgogICAgICAgICAgICBleGl0IDIKICAgICAgICBmaQogICAgZG9uZQp9Cgpj
aGVja19hY2xfbWluaW1hbCAoKSB7CiAgICBsb2NhbCBmaWxlPSIkMSIKICAgIGxvY2FsIG1zZz0i
JHsyOi0nJ30iCiAgICBsb2NhbCBhY2xzCiAgICBhY2xzPSQoZ2V0ZmFjbCAtYyAiJHtmaWxlfSIg
fCBncmVwIC12ICI6OiIpCiAgICBpZiBbIC1uICIkYWNscyIgXQogICAgdGhlbgogICAgICAgIGVj
aG8gIiRtc2ciICJGYWlsLCAnJGZpbGUnIHVuZXhwZWN0ZWQgZ2V0ZmFjbCByZXN1bHQgJyRhY2xz
JyIKICAgICAgICBleGl0IDEKICAgIGZpCn0KCnRlc3RfbW9kaWZ5X29uY2UgKCkgewogICAgbG9j
YWwgZmlsZT0iJDEiCiAgICBsb2NhbCBhY2w9IiQyIgogICAgbG9jYWwgbXNnPSIkezM6LScnfSIK
ICAgICMgZW5zdXJlIHRoYXQgZmlsZSBpcyBwcmlzdGluZQogICAgY2hlY2tfYWNsX21pbmltYWwg
IiRmaWxlIiAiJG1zZyBwcmVjb25kaXRpb246IgogICAgY2hlY2tfb3duZXIgIiRmaWxlIiByb290
ICIkbXNnIHByZWNvbmRpdGlvbjoiCiAgICBjaGVja19ncm91cCAiJGZpbGUiIHJvb3QgIiRtc2cg
cHJlY29uZGl0aW9uOiIKICAgIGlmICEgc2V0ZmFjbCAtbSAiJGFjbCIgIiRmaWxlIgogICAgdGhl
bgogICAgICAgIGVjaG8gIiRtc2ciICAiRmFpbCwgdW5hYmxlIHRvIGNhbGwgc2V0ZmFjbCIKICAg
ICAgICBleGl0IDIKICAgIGZpCiAgICBjaGVja19hY2xfY29udGFpbnMgIiRmaWxlIiAiJGFjbCIg
IiRtc2c6IGFjbCBub3Qgc2V0OiIKICAgIGNoZWNrX293bmVyICIkZmlsZSIgcm9vdCAiJG1zZyBv
d25lciBjb3JydXB0ZWQ6IgogICAgY2hlY2tfZ3JvdXAgIiRmaWxlIiByb290ICIkbXNnIGdyb3Vw
IGNvcnJ1cHRlZDoiCn0KCgp0cmFwICJybSAtcmYgdGVzdGRpciIgRVhJVApta2RpciB0ZXN0ZGly
IHx8IGV4aXQgMQoKCiMgdXNlcgp0b3VjaCB0ZXN0ZGlyL2YxIHx8IGV4aXQgMQpta2RpciB0ZXN0
ZGlyL2QxIHx8IGV4aXQgMQojIHJlZ3VsYXIgZmlsZQp0ZXN0X21vZGlmeV9vbmNlIHRlc3RkaXIv
ZjEgInVzZXI6cm9vdDpyIiAiJExJTkVOTzoiCiMgZGlyZWN0b3J5CnRlc3RfbW9kaWZ5X29uY2Ug
dGVzdGRpci9kMSAidXNlcjpyb290OnIiICIkTElORU5POiIKcm0gLXJmIHRlc3RkaXIvZjEgdGVz
dGRpci9kMQoKI2dyb3VwCnJtIC1yZiB0ZXN0ZGlyL2YxIHRlc3RkaXIvZDEKdG91Y2ggdGVzdGRp
ci9mMSB8fCBleGl0IDEKbWtkaXIgdGVzdGRpci9kMSB8fCBleGl0IDEKIyByZWd1bGFyIGZpbGUK
dGVzdF9tb2RpZnlfb25jZSB0ZXN0ZGlyL2YxICJncm91cDpyb290OnIiICIkTElORU5POiIKIyBk
aXJlY3RvcnkKdGVzdF9tb2RpZnlfb25jZSB0ZXN0ZGlyL2QxICJncm91cDpyb290OnIiICIkTElO
RU5POiIKcm0gLXJmIHRlc3RkaXIvZjEgdGVzdGRpci9kMQoKIyBtdWx0aXBsZSB1c2Vycwp0b3Vj
aCB0ZXN0ZGlyL2YxIHx8IGV4aXQgMQpta2RpciB0ZXN0ZGlyL2QxIHx8IGV4aXQgMQojIHJlZ3Vs
YXIgZmlsZQp0ZXN0X21vZGlmeV9vbmNlIHRlc3RkaXIvZjEgInVzZXI6cm9vdDpyLGdyb3VwOnJv
b3Q6cix1c2VyOmJpbjpydyIgIiRMSU5FTk86IgojIGRpcmVjdG9yeQp0ZXN0X21vZGlmeV9vbmNl
IHRlc3RkaXIvZDEgInVzZXI6cm9vdDpyLGdyb3VwOnJvb3Q6cix1c2VyOmJpbjpydyIgIiRMSU5F
Tk86IgpybSAtcmYgdGVzdGRpci9mMSB0ZXN0ZGlyL2QxCgoKIyBzZXRmYWNsIGRlZmF1bHQgYWNs
cwpta2RpciB0ZXN0ZGlyL2QxIHx8IGV4aXQgMQp0ZXN0X21vZGlmeV9vbmNlIHRlc3RkaXIvZDEg
ImRlZmF1bHQ6dXNlcjpyb290OnIsdXNlcjpyb290OnIiICIkTElORU5POiIKcm0gLXJmIHRlc3Rk
aXIvZDEKCgojIG11bHRpcGxlIGNhbGxzIHRvIHNldGZhY2wgLW0gb24gc2FtZSBmaWxlCnRvdWNo
IHRlc3RkaXIvZjEgfHwgZXhpdCAxCm1rZGlyIHRlc3RkaXIvZDEgfHwgZXhpdCAxCmNoZWNrX293
bmVyIHRlc3RkaXIvZjEgcm9vdCAiJExJTkVOTzogcHJlY29uZGl0aW9uOiIKY2hlY2tfZ3JvdXAg
dGVzdGRpci9mMSByb290ICIkTElORU5POiBwcmVjb25kaXRpb246IgpjaGVja19hY2xfbWluaW1h
bCB0ZXN0ZGlyL2YxICIkTElORU5POiBwcmVjb25kaXRpb246IgoKYWNsMT0idXNlcjpyb290OnIi
CmFjbDI9InVzZXI6YmluOnJ3IgoKaWYgISBzZXRmYWNsIC1tICIkYWNsMSIgdGVzdGRpci9mMSAj
IGZpcnN0IHNldGZhY2wKdGhlbgogICAgZWNobyAiJExJTkVOTzoiICAiRmFpbCwgdW5hYmxlIHRv
IGNhbGwgc2V0ZmFjbCIKICAgIGV4aXQgMgpmaQpjaGVja19hY2xfY29udGFpbnMgdGVzdGRpci9m
MSAiJGFjbDEiICIkTElORU5POiBhY2wxIG5vdCBzZXQ6IgpjaGVja19vd25lciB0ZXN0ZGlyL2Yx
IHJvb3QgIiRMSU5FTk86IG93bmVyIGNvcnJ1cHRlZDoiCmNoZWNrX2dyb3VwIHRlc3RkaXIvZjEg
cm9vdCAiJExJTkVOTzogZ3JvdXAgY29ycnVwdGVkOiIKCmlmICEgc2V0ZmFjbCAtbSAiJGFjbDIi
IHRlc3RkaXIvZjEgIyBzZWNvbmQgc2V0ZmFjbAp0aGVuCiAgICBlY2hvICIkTElORU5POiIgICJG
YWlsLCB1bmFibGUgdG8gY2FsbCBzZXRmYWNsIgogICAgZXhpdCAyCmZpCgpjaGVja19hY2xfY29u
dGFpbnMgdGVzdGRpci9mMSAiJGFjbDEiICIkTElORU5POiBhY2wxIG5vdCBzZXQ6IgpjaGVja19h
Y2xfY29udGFpbnMgdGVzdGRpci9mMSAiJGFjbDIiICIkTElORU5POiBhY2wyIG5vdCBzZXQ6Igpj
aGVja19vd25lciB0ZXN0ZGlyL2YxIHJvb3QgIiRMSU5FTk86IG93bmVyIGNvcnJ1cHRlZDoiCmNo
ZWNrX2dyb3VwIHRlc3RkaXIvZjEgcm9vdCAiJExJTkVOTzogZ3JvdXAgY29ycnVwdGVkOiIKcm0g
LXJmIHRlc3RkaXIvZjEgdGVzdGRpci9kMQoKIyBzZXRmYWNsIHJlY3Vyc2l2ZQp0ZXN0X21vZGlm
eV9yZWN1cnNpdmUgKCkgewogICAgbG9jYWwgcm9vdF9kaXI9IiQxIgogICAgbG9jYWwgYWNsPSIk
MiIKICAgIGxvY2FsIG1zZz0iJHszOi0nJ30iCgogICAgZmluZCAiJHJvb3RfZGlyIiB8IHdoaWxl
IHJlYWQgLXIgZmlsZTsgZG8KICAgICAgICBjaGVja19vd25lciAiJGZpbGUiIHJvb3QgIiRtc2cg
cHJlY29uZGl0aW9uOiIKICAgICAgICBjaGVja19ncm91cCAiJGZpbGUiIHJvb3QgIiRtc2cgcHJl
Y29uZGl0aW9uOiIKICAgICAgICBjaGVja19hY2xfbWluaW1hbCAiJGZpbGUiICIkbXNnIHByZWNv
bmRpdGlvbjoiCiAgICBkb25lCiAgICBpZiAhIHNldGZhY2wgLVIgLW0gIiRhY2wiICIkcm9vdF9k
aXIiCiAgICB0aGVuCiAgICAgICAgZWNobyAiJG1zZyIgICJGYWlsLCB1bmFibGUgdG8gY2FsbCBz
ZXRmYWNsIgogICAgICAgIGV4aXQgMgogICAgZmkKICAgIGZpbmQgIiRyb290X2RpciIgfCB3aGls
ZSByZWFkIC1yIGZpbGU7IGRvCiAgICAgICAgY2hlY2tfb3duZXIgIiRmaWxlIiByb290ICIkbXNn
IG93bmVyIGNvcnJ1cHRlZDoiCiAgICAgICAgY2hlY2tfZ3JvdXAgIiRmaWxlIiByb290ICIkbXNn
IGdyb3VwIGNvcnJ1cHRlZDoiCiAgICAgICAgY2hlY2tfYWNsX2NvbnRhaW5zICIkZmlsZSIgIiRh
Y2wiICIkbXNnIGFjbCBub3Qgc2V0OiIKICAgIGRvbmUKfQoKbWtkaXIgLXAgdGVzdGRpci9kMS9k
MiB8fCBleGl0IDEKdG91Y2ggdGVzdGRpci9kMS9kMi9mMSB8fCBleGl0IDEKdGVzdF9tb2RpZnlf
cmVjdXJzaXZlIHRlc3RkaXIvZDEgInVzZXI6cm9vdDpyLGdyb3VwOnJvb3Q6cix1c2VyOmJpbjpy
dyIgIiRMSU5FTk86IgpybSAtcmYgdGVzdGRpci9kMQoKbWtkaXIgLXAgdGVzdGRpci9kMS9kMiB8
fCBleGl0IDEKbWtkaXIgLXAgdGVzdGRpci9kMS9kMyB8fCBleGl0IDEKdGVzdF9tb2RpZnlfcmVj
dXJzaXZlIHRlc3RkaXIvZDEgImRlZmF1bHQ6dXNlcjpyb290OnJ3eCx1c2VyOnJvb3Q6cixncm91
cDpyb290OnIsdXNlcjpiaW46cnciICIkTElORU5POiIKcm0gLXJmIHRlc3RkaXIvZDEKCiNlY2hv
ICJQYXNzZWQuIgpleGl0IDAK
</data>

          </attachment>
      

    </bug>

</bugzilla>