chiark / gitweb /
core: introduce a new load state "bad-setting"
authorLennart Poettering <lennart@poettering.net>
Fri, 1 Jun 2018 15:46:01 +0000 (17:46 +0200)
committerSven Eden <yamakuzure@gmx.net>
Fri, 24 Aug 2018 14:47:08 +0000 (16:47 +0200)
commitf6d804bc4caa5d0187d672a5f643e5b8cd1980ed
treec038138519abf0064c29eca9a5878d252ab46a39
parentda6d3a3807b6f30d3ae9b76b35069f760244f5b1
core: introduce a new load state "bad-setting"

Since bb28e68477a3a39796e4999a6cbc6ac6345a9159 parsing failures of
certain unit file settings will result in load failures of units. This
introduces a new load state "bad-setting" that is entered in precisely
this case.

With this addition error messages on bad settings should be a lot more
explicit, as we don't have to show some generic "errno" error in that
case, but can explicitly say that a bad setting is at fault.

Internally this unit load state is entered as soon as any configuration
loader call returns ENOEXEC. Hence: config parser calls should return
ENOEXEC now for such essential unit file settings. Turns out, they
generally already do.

Fixes: #9107
src/basic/unit-def.c
src/basic/unit-def.h
src/libelogind/sd-bus/bus-common-errors.c
src/libelogind/sd-bus/bus-common-errors.h