cryptsetup: warn if /etc/crypttab is world-readable
This commit is contained in:
parent
5954c07433
commit
8973790ee6
62
TODO
62
TODO
|
@ -121,8 +121,6 @@ Features:
|
||||||
and we might want to requeue the mounts local-fs acquired through
|
and we might want to requeue the mounts local-fs acquired through
|
||||||
that automatically.
|
that automatically.
|
||||||
|
|
||||||
* neither pkexec nor sudo initialize environ[] from the PAM environment?
|
|
||||||
|
|
||||||
* rework specifier logic so that we can distuingish OOM errors from other errors
|
* rework specifier logic so that we can distuingish OOM errors from other errors
|
||||||
|
|
||||||
* systemd-inhibit: make taking delay locks useful: support sending SIGINT or SIGTERM on PrepareForSleep()
|
* systemd-inhibit: make taking delay locks useful: support sending SIGINT or SIGTERM on PrepareForSleep()
|
||||||
|
@ -131,9 +129,7 @@ Features:
|
||||||
|
|
||||||
* remove any syslog support from log.c -- we probably can't do this before split-off udev is gone for good
|
* remove any syslog support from log.c -- we probably can't do this before split-off udev is gone for good
|
||||||
|
|
||||||
* fedora: connect the timer units of a service to the service via Also= in [Install]
|
* documentation: recommend to connect the timer units of a service to the service via Also= in [Install]
|
||||||
|
|
||||||
* fedora: F20: go timer units all the way, leave cron.daily for cron
|
|
||||||
|
|
||||||
* add a tool that lists active timer units plus their next elapstion and the time the units ran last
|
* add a tool that lists active timer units plus their next elapstion and the time the units ran last
|
||||||
|
|
||||||
|
@ -151,12 +147,6 @@ Features:
|
||||||
|
|
||||||
* man: maybe sort directives in man pages, and take sections from --help and apply them to man too
|
* man: maybe sort directives in man pages, and take sections from --help and apply them to man too
|
||||||
|
|
||||||
* add "# export SYSTEMD_PAGER=" to bash login
|
|
||||||
|
|
||||||
* /usr/bin/service should actually show the new command line
|
|
||||||
|
|
||||||
* fedora: suggest auto-restart on failure, but not on sucess and not on coredump. also, ask people to think about changing the start limit logic. Also point people to RestartPreventExitStatus=, SuccessExitStatus=
|
|
||||||
|
|
||||||
* write UI tool that pops up emergency messages from the journal as notification
|
* write UI tool that pops up emergency messages from the journal as notification
|
||||||
|
|
||||||
* think about window-manager-run-as-user-service problem: exit 0 → activate shutdown.target; exit != 0 → restart service
|
* think about window-manager-run-as-user-service problem: exit 0 → activate shutdown.target; exit != 0 → restart service
|
||||||
|
@ -214,6 +204,9 @@ Features:
|
||||||
- pam: when leaving a session explicitly exclude the ReleaseSession() caller process from the killing spree
|
- pam: when leaving a session explicitly exclude the ReleaseSession() caller process from the killing spree
|
||||||
- logind: GetSessionByPID() should accept 0 as PID value
|
- logind: GetSessionByPID() should accept 0 as PID value
|
||||||
- we should probably handle SIGTERM/SIGINT to not leave dot files around, just in case
|
- we should probably handle SIGTERM/SIGINT to not leave dot files around, just in case
|
||||||
|
- add configuration/switches to use
|
||||||
|
freeze (http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git) and
|
||||||
|
standby (https://bugs.freedesktop.org/show_bug.cgi?id=57793) as suspend modes
|
||||||
|
|
||||||
* exec: when deinitializating a tty device fix the perms and group, too, not only when initializing. Set access mode/gid to 0620/tty.
|
* exec: when deinitializating a tty device fix the perms and group, too, not only when initializing. Set access mode/gid to 0620/tty.
|
||||||
|
|
||||||
|
@ -239,7 +232,6 @@ Features:
|
||||||
- check if we can make journalctl by default use --follow mode inside of less if called without args?
|
- check if we can make journalctl by default use --follow mode inside of less if called without args?
|
||||||
- maybe add API to send pairs of iovecs via sd_journal_send
|
- maybe add API to send pairs of iovecs via sd_journal_send
|
||||||
- journal: when writing journal auto-rotate if time jumps backwards
|
- journal: when writing journal auto-rotate if time jumps backwards
|
||||||
- gatewayd: should run under its own UID
|
|
||||||
- journal: add a setgid "systemd-journal" utility to invoke from libsystemd-journal, which passes fds via STDOUT and does PK access
|
- journal: add a setgid "systemd-journal" utility to invoke from libsystemd-journal, which passes fds via STDOUT and does PK access
|
||||||
- journactl: support negative filtering, i.e. FOOBAR!="waldo",
|
- journactl: support negative filtering, i.e. FOOBAR!="waldo",
|
||||||
and !FOOBAR for events without FOOBAR.
|
and !FOOBAR for events without FOOBAR.
|
||||||
|
@ -248,7 +240,6 @@ Features:
|
||||||
- journal-send.c, log.c: when the log socket is clogged, and we drop, count this and write a message about this when it gets unclogged again.
|
- journal-send.c, log.c: when the log socket is clogged, and we drop, count this and write a message about this when it gets unclogged again.
|
||||||
- journal: find a way to allow dropping history early, based on priority, other rules
|
- journal: find a way to allow dropping history early, based on priority, other rules
|
||||||
- journal: When used on NFS, check payload hashes
|
- journal: When used on NFS, check payload hashes
|
||||||
- journal: When used on NFS make sure wake up sd_journal_wait() every 2s, to handle missing inotify
|
|
||||||
- Introduce journalctl -b <nr> to show journal messages of a previous boot
|
- Introduce journalctl -b <nr> to show journal messages of a previous boot
|
||||||
- journald: check whether it is OK if the client can still modify delivered journal entries
|
- journald: check whether it is OK if the client can still modify delivered journal entries
|
||||||
- journal live copy, based on libneon (client) and libmicrohttpd (server)
|
- journal live copy, based on libneon (client) and libmicrohttpd (server)
|
||||||
|
@ -259,7 +250,6 @@ Features:
|
||||||
- journalctl: show multiline log messages sanely, expand tabs, and show all valid utf8 messages
|
- journalctl: show multiline log messages sanely, expand tabs, and show all valid utf8 messages
|
||||||
- journal: store euid in journal if it differs from uid
|
- journal: store euid in journal if it differs from uid
|
||||||
- journal: sanely deal with entries which are larger than the individual file size, but where the components would fit
|
- journal: sanely deal with entries which are larger than the individual file size, but where the components would fit
|
||||||
- journalctl: make journalctl smarter, and actually check groups that have access to /var/log/journal before printing message about recomming group membership for journal access
|
|
||||||
- Replace utmp, wtmp, btmp, and lastlog completely with journal
|
- Replace utmp, wtmp, btmp, and lastlog completely with journal
|
||||||
- Port upower to use the journal for historical power information used in future calculations
|
- Port upower to use the journal for historical power information used in future calculations
|
||||||
|
|
||||||
|
@ -288,9 +278,6 @@ Features:
|
||||||
- systemctl: "Journal has been rotated since unit was started." message is misleading
|
- systemctl: "Journal has been rotated since unit was started." message is misleading
|
||||||
- support "systemctl stop foobar@.service" to stop all units matching a certain template
|
- support "systemctl stop foobar@.service" to stop all units matching a certain template
|
||||||
- Something is wrong with symlink handling of "autovt@.service" in "systemctl list-unit-files"
|
- Something is wrong with symlink handling of "autovt@.service" in "systemctl list-unit-files"
|
||||||
- add configuration/switches to use
|
|
||||||
freeze (http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git) and
|
|
||||||
standby (https://bugs.freedesktop.org/show_bug.cgi?id=57793) as suspend modes
|
|
||||||
|
|
||||||
* introduce ntp.service (or suchlike) as symlink that is used to arbitrate between various NTP implementations
|
* introduce ntp.service (or suchlike) as symlink that is used to arbitrate between various NTP implementations
|
||||||
|
|
||||||
|
@ -363,8 +350,6 @@ Features:
|
||||||
|
|
||||||
* currently x-systemd.timeout is lost in the initrd, since crypttab is copied into dracut, but fstab isn't
|
* currently x-systemd.timeout is lost in the initrd, since crypttab is copied into dracut, but fstab isn't
|
||||||
|
|
||||||
* WorkingDirectory: support env var replacements like in ExecStart= so that people can use $HOME
|
|
||||||
|
|
||||||
* refuse boot if /etc/machine-id is not useful (or set taint?)
|
* refuse boot if /etc/machine-id is not useful (or set taint?)
|
||||||
|
|
||||||
* nspawn:
|
* nspawn:
|
||||||
|
@ -385,7 +370,6 @@ Features:
|
||||||
- cryptsetup-generator: allow specification of passwords in crypttab itself
|
- cryptsetup-generator: allow specification of passwords in crypttab itself
|
||||||
- move cryptsetup key caching into kernel keyctl?
|
- move cryptsetup key caching into kernel keyctl?
|
||||||
https://bugs.freedesktop.org/show_bug.cgi?id=54982
|
https://bugs.freedesktop.org/show_bug.cgi?id=54982
|
||||||
- when key file cannot be found, read it from kbd in cryptsetup
|
|
||||||
|
|
||||||
* move debug shell to tty6 and make sure this doesn't break the gettys on tty6
|
* move debug shell to tty6 and make sure this doesn't break the gettys on tty6
|
||||||
|
|
||||||
|
@ -454,14 +438,10 @@ Features:
|
||||||
|
|
||||||
* introduce Type=pid-file
|
* introduce Type=pid-file
|
||||||
|
|
||||||
* maybe allow services with ExecStop= set, but no ExecStart=?
|
|
||||||
|
|
||||||
* change Requires=basic.target to RequisiteOverride=basic.target
|
* change Requires=basic.target to RequisiteOverride=basic.target
|
||||||
|
|
||||||
* support rd.luks.allow-discards= kernel cmdline params in cryptsetup generator
|
* support rd.luks.allow-discards= kernel cmdline params in cryptsetup generator
|
||||||
|
|
||||||
* drop accountsservice's StandardOutput=syslog and Type=dbus fields
|
|
||||||
|
|
||||||
* when breaking cycles drop sysv services first, then services from /run, then from /etc, then from /usr
|
* when breaking cycles drop sysv services first, then services from /run, then from /etc, then from /usr
|
||||||
|
|
||||||
* move passno parsing to fstab generator
|
* move passno parsing to fstab generator
|
||||||
|
@ -492,8 +472,6 @@ Features:
|
||||||
|
|
||||||
* cleanup syslog 'priority' vs. 'level' wording
|
* cleanup syslog 'priority' vs. 'level' wording
|
||||||
|
|
||||||
* dbus upstream still refers to dbus.target and shouldn't
|
|
||||||
|
|
||||||
* when a service has the same env var set twice we actually store it twice and return that in systemctl show -p... We should only show the last setting
|
* when a service has the same env var set twice we actually store it twice and return that in systemctl show -p... We should only show the last setting
|
||||||
|
|
||||||
* support container_ttys=
|
* support container_ttys=
|
||||||
|
@ -508,8 +486,6 @@ Features:
|
||||||
|
|
||||||
* default unix qlen is too small (10). bump sysctl? add sockopt?
|
* default unix qlen is too small (10). bump sysctl? add sockopt?
|
||||||
|
|
||||||
* dbus: in fedora, make /var/lib/dbus/machine-id a symlink to /etc/machine-id
|
|
||||||
|
|
||||||
* save coredump in Windows/Mozilla minidump format
|
* save coredump in Windows/Mozilla minidump format
|
||||||
|
|
||||||
* support crash reporting operation modes (https://live.gnome.org/GnomeOS/Design/Whiteboards/ProblemReporting)
|
* support crash reporting operation modes (https://live.gnome.org/GnomeOS/Design/Whiteboards/ProblemReporting)
|
||||||
|
@ -592,8 +568,6 @@ Features:
|
||||||
|
|
||||||
* support systemd.mask= on the kernel command line.
|
* support systemd.mask= on the kernel command line.
|
||||||
|
|
||||||
* reuse mkdtemp namespace dirs in /tmp?
|
|
||||||
|
|
||||||
* recreate systemd's D-Bus private socket file on SIGUSR2
|
* recreate systemd's D-Bus private socket file on SIGUSR2
|
||||||
|
|
||||||
* Support --test based on current system state
|
* Support --test based on current system state
|
||||||
|
@ -624,6 +598,13 @@ Features:
|
||||||
works with ^C
|
works with ^C
|
||||||
- add documentation to systemd.daemon
|
- add documentation to systemd.daemon
|
||||||
|
|
||||||
|
* bootchart:
|
||||||
|
- plot per-process IO utilization
|
||||||
|
- group processes based on service association (cgroups)
|
||||||
|
- document initcall_debug
|
||||||
|
- put bootcharts in the journal
|
||||||
|
- kernel cmdline "bootchart" option for simplicity?
|
||||||
|
|
||||||
External:
|
External:
|
||||||
|
|
||||||
* dbus:
|
* dbus:
|
||||||
|
@ -649,12 +630,21 @@ External:
|
||||||
|
|
||||||
* kernel: add device_type = "fb", "fbcon" to class "graphics"
|
* kernel: add device_type = "fb", "fbcon" to class "graphics"
|
||||||
|
|
||||||
* bootchart:
|
* drop accountsservice's StandardOutput=syslog and Type=dbus fields
|
||||||
- plot per-process IO utilization
|
|
||||||
- group processes based on service association (cgroups)
|
* dbus upstream still refers to dbus.target and shouldn't
|
||||||
- document initcall_debug
|
|
||||||
- put bootcharts in the journal
|
* dbus: in fedora, make /var/lib/dbus/machine-id a symlink to /etc/machine-id
|
||||||
- kernel cmdline "bootchart" option for simplicity?
|
|
||||||
|
* add "# export SYSTEMD_PAGER=" to bash login
|
||||||
|
|
||||||
|
* /usr/bin/service should actually show the new command line
|
||||||
|
|
||||||
|
* fedora: suggest auto-restart on failure, but not on sucess and not on coredump. also, ask people to think about changing the start limit logic. Also point people to RestartPreventExitStatus=, SuccessExitStatus=
|
||||||
|
|
||||||
|
* fedora: F20: go timer units all the way, leave cron.daily for cron
|
||||||
|
|
||||||
|
* neither pkexec nor sudo initialize environ[] from the PAM environment?
|
||||||
|
|
||||||
Regularly:
|
Regularly:
|
||||||
|
|
||||||
|
|
|
@ -328,13 +328,13 @@ static int parse_proc_cmdline(char ***arg_proc_cmdline_disks, char **arg_proc_cm
|
||||||
}
|
}
|
||||||
|
|
||||||
int main(int argc, char *argv[]) {
|
int main(int argc, char *argv[]) {
|
||||||
|
_cleanup_strv_free_ char **arg_proc_cmdline_disks_done = NULL;
|
||||||
|
_cleanup_strv_free_ char **arg_proc_cmdline_disks = NULL;
|
||||||
|
_cleanup_free_ char *arg_proc_cmdline_keyfile = NULL;
|
||||||
_cleanup_fclose_ FILE *f = NULL;
|
_cleanup_fclose_ FILE *f = NULL;
|
||||||
unsigned n = 0;
|
unsigned n = 0;
|
||||||
int r = EXIT_SUCCESS;
|
int r = EXIT_SUCCESS;
|
||||||
char **i;
|
char **i;
|
||||||
_cleanup_strv_free_ char **arg_proc_cmdline_disks_done = NULL;
|
|
||||||
_cleanup_strv_free_ char **arg_proc_cmdline_disks = NULL;
|
|
||||||
_cleanup_free_ char *arg_proc_cmdline_keyfile = NULL;
|
|
||||||
|
|
||||||
if (argc > 1 && argc != 4) {
|
if (argc > 1 && argc != 4) {
|
||||||
log_error("This program takes three or no arguments.");
|
log_error("This program takes three or no arguments.");
|
||||||
|
@ -357,8 +357,9 @@ int main(int argc, char *argv[]) {
|
||||||
return EXIT_SUCCESS;
|
return EXIT_SUCCESS;
|
||||||
|
|
||||||
if (arg_read_crypttab) {
|
if (arg_read_crypttab) {
|
||||||
f = fopen("/etc/crypttab", "re");
|
struct stat st;
|
||||||
|
|
||||||
|
f = fopen("/etc/crypttab", "re");
|
||||||
if (!f) {
|
if (!f) {
|
||||||
if (errno == ENOENT)
|
if (errno == ENOENT)
|
||||||
r = EXIT_SUCCESS;
|
r = EXIT_SUCCESS;
|
||||||
|
@ -366,7 +367,20 @@ int main(int argc, char *argv[]) {
|
||||||
r = EXIT_FAILURE;
|
r = EXIT_FAILURE;
|
||||||
log_error("Failed to open /etc/crypttab: %m");
|
log_error("Failed to open /etc/crypttab: %m");
|
||||||
}
|
}
|
||||||
} else for (;;) {
|
|
||||||
|
goto next;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (fstat(fileno(f), &st) < 0) {
|
||||||
|
log_error("Failed to stat /etc/crypttab: %m");
|
||||||
|
r = EXIT_FAILURE;
|
||||||
|
goto next;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (st.st_mode & 0005)
|
||||||
|
log_warning("/etc/crypttab is world-readable. This is usually not a good idea.");
|
||||||
|
|
||||||
|
for (;;) {
|
||||||
char line[LINE_MAX], *l;
|
char line[LINE_MAX], *l;
|
||||||
_cleanup_free_ char *name = NULL, *device = NULL, *password = NULL, *options = NULL;
|
_cleanup_free_ char *name = NULL, *device = NULL, *password = NULL, *options = NULL;
|
||||||
int k;
|
int k;
|
||||||
|
@ -420,6 +434,7 @@ int main(int argc, char *argv[]) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
next:
|
||||||
STRV_FOREACH(i, arg_proc_cmdline_disks) {
|
STRV_FOREACH(i, arg_proc_cmdline_disks) {
|
||||||
/*
|
/*
|
||||||
Generate units for those UUIDs, which were specified
|
Generate units for those UUIDs, which were specified
|
||||||
|
|
|
@ -48,6 +48,8 @@ extern "C" {
|
||||||
*
|
*
|
||||||
* - enforce alignment of pointers passed in
|
* - enforce alignment of pointers passed in
|
||||||
* - negotiation for attach attributes
|
* - negotiation for attach attributes
|
||||||
|
*
|
||||||
|
* - for kernel and unix transports allow setting the unix user/access mode for the node
|
||||||
*/
|
*/
|
||||||
|
|
||||||
typedef struct sd_bus sd_bus;
|
typedef struct sd_bus sd_bus;
|
||||||
|
|
Loading…
Reference in New Issue