Skip to contentSkip to conversation

docs/manual: makedevs only accepts numerical values

PATCH New Yann E. MORIN 2 participants 2 messages Aug 28, 2015 – Aug 29, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 docs/manual: makedevs only accepts numerical values

Because of a limitation in how makedevs resolves names [0], usernames and groupnames would be resolved with UIDs and GIDs from the host, not the target.

Fixing makedevs would involve us rewriting a /etc/passwd and /etc/group parser (not too complex, still not trivial).

Document that the user and group fields should only be numbers, not names. Also, the manual is currently mislieading, as an example indeed uses names for the user and group fields. Fix that and expand the explanations.

The mode field should also be numbers, not a rwx combo.

Slight typographical eye-candy about major/minor fields.

[0] it uses getpwnma(3) and getgrnam(3) to resolve names to IDs, and those only resolve names on the host (i.e. there is no way to specify a chroot-like feature)

---
Note: fixing makedevs to support names would require us to provide replacements for getpwnam() and getgrnam(), that can read from an alternate passwd and group file. This is not too complex, but far from trivial. Fixing the docs is much easier, that close to the release...

---
 docs/manual/makedev-syntax.txt | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)
docs/manual/makedev-syntax.txt +7 -4
index 283119d..cae37f7 100644
@@ -28,8 +28,9 @@ There are a few non-trivial blocks:
   * c: a character device file
   * b: a block device file
   * p: a named pipe
-- +mode+, +uid+ and +gid+ are the usual permissions settings
-- +major+ and +minor+ are here for device files - set to - for other
+- +mode+, +uid+ and +gid+ are the usual permissions settings (only
+  numerical values are allowed)
+- +major+ and +minor+ are here for device files, set to +-+ for other
   files
 - +start+, +inc+ and +count+ are for when you want to create a batch
   of files, and can be reduced to a loop, beginning at +start+,
@@ -43,10 +44,12 @@ this syntax, you will need to put:
 ----
 
 Alternatively, if you want to change owner/permission of a directory
-recursively, you can put:
+recursively, you can put (to set UID to 123, GID to 456 and access
+rights to rwxr-x--- for the directory /usr/share/myapp and all files
+and directories below it):
 
 ----
-/usr/share/myapp r 750 myuser myuser - - - - -
+/usr/share/myapp r 750 123 456 - - - - -
 ----
 
 On the other hand, if you want to create the device file +/dev/hda+
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 docs/manual: makedevs only accepts numerical values

Dear Yann E. MORIN,

Quoted 9 lines · Yann E. MORIN

Because of a limitation in how makedevs resolves names [0], usernames and groupnames would be resolved with UIDs and GIDs from the host, not the target.

Fixing makedevs would involve us rewriting a /etc/passwd and /etc/group parser (not too complex, still not trivial).

Document that the user and group fields should only be numbers, not names. Also, the manual is currently mislieading, as an example indeed

mislieading -> misleading typo fixed.

And patch applied to master, thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com