Skip to content
Snippets Groups Projects
Commit 2eaca825 authored by Jason A. Donenfeld's avatar Jason A. Donenfeld
Browse files

Use pipefail and randomize intermediate encrypted.


Matthew writes:

    If the initial decrypt fails then the rest of the line shouldn't
    continue, as it won't be a properly decrypted password being
    re-encrypted and written over the existing passfile.

    One solution to this would be to enable pipefail (set -o pipefail) -
    either just before, or at the start of this script.  This would
    cause the failure of any of the commands in a pipe to set the return
    status of the whole pipeline to non-zero (the last failed command's
    return code is used).

We take his suggestion with this patch. While we're at it, we take a
little bit extra care (though not too much extra care) to select a more
random intermediate password, in case folks have a strange habit of
using a dot-new extension on files.

Suggested-by: default avatarMatthew Richardson <m.richardson@ed.ac.uk>
Signed-off-by: default avatarJason A. Donenfeld <Jason@zx2c4.com>
parent cca731a2
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment