tmpfiles: rework file attribute code
authorLennart Poettering <lennart@poettering.net>
Wed, 8 Apr 2015 20:35:52 +0000 (22:35 +0200)
committerLennart Poettering <lennart@poettering.net>
Wed, 8 Apr 2015 20:35:52 +0000 (22:35 +0200)
commit88ec4dfa289cd97496dbb9670365a3d4be13d41c
treeb1c13294254baf1c43a951d2f5519159ee29f6f7
parent1ed8f8c16da99551c6731764759878a0bc243fde
tmpfiles: rework file attribute code

- Stick to one type for the flags field: unsigned. This appears to be
  what the kernel uses, and there's no point in using something else.

- compress the flags array by avoiding sparse entries

- extend some error messages to not use abbreviated words

- avoid TTOCTTOU issues by invoking fstat() after open() when applying
  file flags

- add explanation why we need to check the file type with fstat().

- don't needlessly abbreviate "attribute" as "attrib", in particually as
  "chattr" abbreviates it as "attr" rather than "attrib".
src/tmpfiles/tmpfiles.c