Skip to content

[19.03] backport Fix docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid#40562

Merged
thaJeztah merged 1 commit into
moby:19.03from
thaJeztah:19.03_backport_39353_subgid_subuid
Apr 2, 2020
Merged

[19.03] backport Fix docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid#40562
thaJeztah merged 1 commit into
moby:19.03from
thaJeztah:19.03_backport_39353_subgid_subuid

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Feb 22, 2020

Copy link
Copy Markdown
Member

This fix tries to address the issue raised in #39353 where
docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid.

The issue was that, mapping to /etc/sub[u,g]id in docker does not
allow numeric ID.

This fix fixes the issue by probing other combinations (uid:groupname, username:gid, uid:gid)
when normal username:groupname fails.

This fix fixes #39353.

- Description for the changelog


Backport of #39764

… /etc/subgid

This fix tries to address the issue raised in 39353 where
docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid.

The issue was that, mapping to `/etc/sub[u,g]id` in docker does not
allow numeric ID.

This fix fixes the issue by probing other combinations (uid:groupname, username:gid, uid:gid)
when normal username:groupname fails.

This fix fixes 39353.

Signed-off-by: Yong Tang <[email protected]>
(cherry picked from commit f09dc2f)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 19.03.8 milestone Feb 22, 2020
@cpuguy83 cpuguy83 changed the title [19.03 backport Fix docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid [19.03] backport Fix docker crash when creating namespaces with UID in /etc/subuid and /etc/subgid Mar 3, 2020

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah modified the milestones: 19.03.8, 19.03.9 Mar 11, 2020
@thaJeztah
thaJeztah merged commit 89f296a into moby:19.03 Apr 2, 2020
@thaJeztah

Copy link
Copy Markdown
Member Author

ah, crap.

merged this one just as GitHub was having issues, so now it's merged, but still open, and no way to close it?

@thaJeztah
thaJeztah deleted the 19.03_backport_39353_subgid_subuid branch April 2, 2020 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants