From 3cae66d2b4a3eba94cb0765db85ff5cdc0ac194a Mon Sep 17 00:00:00 2001
From: "Alan J. Pippin" <alan@pippins.net>
Date: Mon, 3 Feb 2020 14:01:29 -0700
Subject: [PATCH] Fixed bug that caused empty error messages to be emailed back

Change-Id: I28685c64435343bc48c938cb422ad06373de01f6
---
 zfs-replicate-all | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/zfs-replicate-all b/zfs-replicate-all
index 66949df..f5e23e7 100755
--- a/zfs-replicate-all
+++ b/zfs-replicate-all
@@ -50,16 +50,15 @@ cleanup_and_exit() {
 }
 trap cleanup_and_exit INT
 
-fatal() {
+fatal_and_exit() {
   # echo message to terminal
   echo -e 2>&1 "$1"
   # send email notification
-  echo -e "$1" | $mailx -s "zfs-replicate-all on $hostname failed" "$mailto"
-}
-
-fatal_and_exit() {
-  fatal $*
-  exit 1
+  echo -e "$1" | $mailx -s "zfs-replicate-all on $hostname failed" "$2"
+  # exit with bad return code unless 3rd argument was defined that says not to
+  if [[ -z "$3" ]]; then
+    exit 1
+  fi
 }
 
 # This function executes the replicate command and checks the stoptime
@@ -153,7 +152,7 @@ if [[ $import_export_backup_pool == 1 ]]; then
     zpool export $backup_pool
     if [ $? != 0 ]; then
       lsof /$backup_pool/*
-      fatal "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto"
+      fatal_and_exit "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto" 0
     fi
   fi
 fi
-- 
2.34.1