df: don't output control characters in a mount point name
authorPádraig Brady <P@draigBrady.com>
Fri, 13 Jul 2012 01:27:26 +0000 (02:27 +0100)
committerPádraig Brady <P@draigBrady.com>
Mon, 16 Jul 2012 01:48:31 +0000 (02:48 +0100)
commit3ed70fd559c3fbed8383b50373e6d23d1857dc52
tree89b15d9f2f62a6263993183648624b5b57a922f6
parentac00d23e1a90dc5a8cd0f6de0e61eb401d5089d5
df: don't output control characters in a mount point name

It's awkward to read and problematic for scripts when
control characters like '\n' are output.

Note other fields are already handled with mbsalign,
which converts non printable chars to the replacement char.
A caveat to note with that, is the replacement char takes
a place in the field and so possibly truncates the field
if it was the widest field in the records.

Note a more general replacement function, that
handles all printable, or non white space characters,
would require more sophisticated support for various
encodings, and the complexity vs benefit was not
deemed beneficial enough at present.
Perhaps in future a more general replacement function
could be shared between the various utilities.

Note <space> is unaffected in any field,
which could impact scripts processing the output.
However any of the number fields at least could have
spaces considering `LANG=fr_FR df -B\'1`, so it's
probably best to leave spaces, which also allows
scripts to handle mount points with spaces without change.

* src/df.c (hide_problematic_chars): Replace control chars with '?'.
* tests/df/problematic-chars: Add a new root only test.
* tests/Makefile.am: Reference the new test.
* NEWS: Mention the fix.
NEWS
src/df.c
tests/Makefile.am
tests/df/problematic-chars [new file with mode: 0755]