cryptsetup: rework how we log about activation failures v239-23
authorLennart Poettering <lennart@poettering.net>
Mon, 21 Jan 2019 19:20:35 +0000 (20:20 +0100)
committerThe Plumber <50238977+systemd-rhel-bot@users.noreply.github.com>
Wed, 18 Dec 2019 12:25:41 +0000 (13:25 +0100)
commit966ecf0011a02c7823083a7868b8589fdf850be8
tree61d9f2ecd1a4dbcaeb844bdbf7b51148325e262d
parent3bbacfb22a9266769a41dee6f8f594fbeb6287fc
cryptsetup: rework how we log about activation failures

First of all let's always log where the errors happen, and not in an
upper stackframe, in all cases. Previously we'd do this somethis one way
and sometimes another, which resulted in sometimes duplicate logging and
sometimes none.

When we cannot activate something due to bad password the kernel gives
us EPERM. Let's uniformly return this EAGAIN, so tha the next password
is tried. (previously this was done in most cases but not in all)

When we get EPERM let's also explicitly indicate that this probably
means the password is simply wrong.

Fixes: #11498
(cherry picked from commit 6f177c7dc092eb68762b4533d41b14244adb2a73)

Related: #1776408
src/cryptsetup/cryptsetup.c