From 291948a54de3d3e636ce557e3fad4b125bb793bd Mon Sep 17 00:00:00 2001 From: Scott Worley Date: Sat, 21 Oct 2017 21:01:23 -0700 Subject: [PATCH] Error messages on stderr --- overonion | 20 ++++++++++---------- overonion-make-key | 8 ++++---- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/overonion b/overonion index 600dafb..a33351a 100755 --- a/overonion +++ b/overonion @@ -1,28 +1,28 @@ #!/bin/bash -if (( $# != 2));then - echo "usage: overonion e|d keyfile" +function die() { + echo "$*" >&2 exit 1 +} + +if (( $# != 2));then + die "usage: overonion e|d keyfile" fi mode=$1 if [[ "$mode" != e && "$mode" != d ]];then - echo "Use 'e' for encrypt or 'd' for decrypt" - exit 1 + die "Use 'e' for encrypt or 'd' for decrypt" fi keyfile=$2 if [[ ! -e "$keyfile" ]];then - echo "Keyfile not found" - exit 1 + die "Keyfile not found" fi if [[ ! -r "$keyfile" ]];then - echo "Cannot read keyfile" - exit 1 + die "Cannot read keyfile" fi num_layers=$(wc -l < "$keyfile") if (( num_layers < 20 ));then - echo "Keyfile doesn't have enough layers to be an onion" - exit 1 + die "Keyfile doesn't have enough layers to be an onion" fi if [[ "$mode" == e ]];then diff --git a/overonion-make-key b/overonion-make-key index 6cfc349..da564ad 100755 --- a/overonion-make-key +++ b/overonion-make-key @@ -24,18 +24,18 @@ if [[ "$1" == '--make_INSECURE_key' ]];then fi if (( $# != 1));then - echo "usage: overonion-make-key keyfile" + echo "usage: overonion-make-key keyfile" >&2 exit 1 fi keyfile=$1 if [[ -e "$keyfile" ]];then - echo "That keyfile already exists. I refuse to overwrite it." + echo "That keyfile already exists. I refuse to overwrite it." >&2 exit 1 fi i=0 while read -r cipher;do - echo -n $'\r'"Generating key $((++i))/${#ciphers[*]}" + echo -n $'\r'"Generating key $((++i))/${#ciphers[*]}" >&2 cat >> "$keyfile" <<< "$cipher $(head -c 99 "$random_source" | base64 --wrap=0 )" done < <( IFS=$'\n'; shuf <<< "${ciphers[*]}"; ) -echo +echo >&2 -- 2.44.1