nspawn: consistently fail if parsing the environment fails
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Fri, 27 Sep 2019 11:58:06 +0000 (13:58 +0200)
committerMichael Biebl <biebl@debian.org>
Fri, 15 Nov 2019 21:01:17 +0000 (21:01 +0000)
commit6a0b5cbec68ca5e1e10531298a111deba8dbd7f7
tree2d488318b687195c0cffda41c668786817594bb9
parent1dc35388c8b6f6ad8bec7bd565ca04d4b8a27a16
nspawn: consistently fail if parsing the environment fails

We would parse the environment twice (to re-apply settings after reading
config from disk), but we would not check the return code first time.
This means that for some settings we would ignore invalid values, while
for others, we'd fail at some point.

Let's just consistently fail. Those environment variables define important
aspects of behaviour, and it is better for the user if we ignore invalid
values. (Unknown settings are still ignored, so forward compatibility is
maintained.)

(cherry picked from commit 490486842b6521104b594c511ca912504f61a8f3)
(cherry picked from commit 9d8e889810b50b5be6eeecec637e4a8bdda30977)

Gbp-Pq: Name nspawn-consistently-fail-if-parsing-the-environment-fails.patch
src/nspawn/nspawn.c