]> git.ipfire.org Git - thirdparty/systemd.git/commit
fsck: copy out device argument from argv[] before forking
authorLennart Poettering <lennart@poettering.net>
Thu, 28 Mar 2019 18:26:21 +0000 (19:26 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Fri, 29 Mar 2019 07:18:25 +0000 (08:18 +0100)
commitbd169c2be0fbdaf6eb2ea7951e650d5e5983fbf6
tree177a32bcdfe84ffca2491f18f1b8905d0de91828
parent7232c1f9da0904de479b17dc58410b972639bb4e
fsck: copy out device argument from argv[] before forking

We nowadays rename our child processes, hence argv[] will be clobbered,
let's hence copy the device path to dynamic memory before forking.

This is fall-out from 60ffa37a65a96c3af857a3dfc4a6fd47b20cc90e since we
now a lot more often end up overriding the argv[] buffer than before,
simple because we know what to override.

These kind of bugs kinda suck. THere are only two options here: stop
overriding argv[] for all cases (or just these cases) or explicitly
copying out everything we need in child processes before forking. With
this patch I opt for the latter, though I am not 100% convinced this is
a great solution. Just a better solution than everything else, i.e.
allowing argv[] to remain out of sync with what others see.

Fixes: #12135
src/fsck/fsck.c