Bug 13996 - Parsing of user:group for start-stop-daemon fails if username contains a period
Summary: Parsing of user:group for start-stop-daemon fails if username contains a period
Status: NEW
Alias: None
Product: Busybox
Classification: Unclassified
Component: Standard Compliance (show other bugs)
Version: 1.33.x
Hardware: All Linux
: P5 normal
Target Milestone: ---
Assignee: unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-11 18:38 UTC by Glenn Burkhardt
Modified: 2021-07-11 18:38 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Glenn Burkhardt 2021-07-11 18:38:08 UTC 
We typically use periods in user and group names.  The function parse_chown_usergroup_or_die() supports an obsolete delimiter for separating the user/group name.  For 'start-stop-daemon', the documentation clearly calls for ':' to be the delimiter.

See ./libpwdgrp/uidgid_get.c line 96:

95              /* Check if there is a group name */
96              group = strchr(user_group, '.'); /* deprecated? */
97              if (!group)

Yes, it's deprecated, and should be removed.