Bug 191611
| Summary: | getmntent(3): does not state /etc/fstab quoting rules correctly | ||
|---|---|---|---|
| Product: | Documentation | Reporter: | anders |
| Component: | man-pages | Assignee: | documentation_man-pages (documentation_man-pages) |
| Status: | RESOLVED CODE_FIX | ||
| Severity: | normal | CC: | mtk.manpages |
| Priority: | P1 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Kernel Version: | Subsystem: | ||
| Regression: | No | Bisected commit-id: | |
Thanks. I made the text:
Since fields in the mtab and fstab files are separated by white‐
space, octal escapes are used to represent the characters space
(\040), tab (\011), newline (\012), and backslash (\\) in those
files when they occur in one of the four strings in a mntent
structure. The routines addmntent() and getmntent() will convert
from string representation to escaped representation and back.
When converting from escaped representation, the sequence \134 is
also converted to a backslash.
I think this fixes your issue, so I'm closing this bug. Please reopen if you see something still to fix.
|
The current man page says: "Since fields in the mtab and fstab files are separated by whitespace, octal escapes are used to represent the four characters space (\040), tab (\011), newline (\012) and backslash (\134) in those files when they occur in one of the four strings in a mntent structure. The routines addmntent() and getmntent() will convert from string representation to escaped representation and back." * Problem: The currrent source of glibc (2.24, file misc/mntent_r.c) does things slightly differently: It encodes \ (backslash) into '\\', not into '\134'. it decodes both '\\' and '\134' into \ (backslash). It looks very much like the currently preferred encoding of backslash is '\\', as that's the only one that works both ways. * Suggestion: Change '(\134)' in the cited passage to '(\\)', and add a note after the cited paragraph that '\134' is also allowed on reading. Note: ('\134' is an odd octal escape, as it doesn't start with '\0' ...)