From ec82f965da3cf4853e56f2abcc42b7e7045f6530 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Fonseca?= <jrfonseca@users.sourceforge.net>
Date: Wed, 8 Aug 2007 15:53:42 +0000
Subject: Minor corrections.

---
 esmtp-wrapper | 35 +++++++++++++++++------------------
 1 file changed, 17 insertions(+), 18 deletions(-)
 mode change 100644 => 100755 esmtp-wrapper

diff --git a/esmtp-wrapper b/esmtp-wrapper
old mode 100644
new mode 100755
index 4215c4b..c090465
--- a/esmtp-wrapper
+++ b/esmtp-wrapper
@@ -22,32 +22,32 @@ mktemp="/bin/mktemp"
 
 queue_mail() { # ($@)
 	local ret=0
-	mkdir -p $qdir || {
-		echo "unable to create queue dir $qdir"
+	mkdir -p "$qdir" || {
+		echo "unable to create queue dir $qdir" 2>&1
 		return 1
 	}
-	mdir="`$mktemp -d $qdir/XXXXXXXX`"
+	mdir="`$mktemp -d "$qdir/XXXXXXXX"`"
 	[ -d "$mdir" ] || {
-		echo "unable to create tempdir inside $qdir"
+		echo "unable to create tempdir inside $qdir" 2>&1
 		return 1
 	}
 
-	echo "$@" >$mdir/cmd || ret=1
-	cat </dev/stdin >$mdir/mail || ret=1
+	echo "$@" >"$mdir/cmd" || ret=1
+	cat </dev/stdin >"$mdir/mail" || ret=1
 	[ $ret -eq 0 ] || {
-		echo "could not enqueue mail in dir $mdir"
+		echo "could not enqueue mail in dir $mdir" 2>&1
 		return 1
 	}
 
-	chmod 0600 $mdir/* || echo "warning setting secure permissions failed!"
+	chmod 0600 $mdir/* || echo "warning setting secure permissions failed!" 2>&1
 	return 0
 }
 
 show_mail() { # ($mdir)
 	echo "mail in dir $1:"
-	printf '\t%s' "`grep ^From: $1/mail`"
-	printf '\t%s' "`grep ^To: $1/mail`"
-	printf '\t%s\n' "`grep ^Date: $1/mail`"
+	printf '\t%s' "`grep ^From: "$1/mail"`"
+	printf '\t%s' "`grep ^To: "$1/mail"`"
+	printf '\t%s\n' "`grep ^Date: "$1/mail"`"
 }
 
 show_queue() { # ()
@@ -60,10 +60,9 @@ show_queue() { # ()
 	echo "$i mails to deliver"
 }
 
-
 send_mail() { # ($mdir)
-	$esmtp $(<$1/cmd) <$1/mail && \
-		rm -rf $1 || return 1
+	$esmtp $(<"$1/cmd") <"$1/mail" && \
+		rm -rf "$1" || return 1
 	return $?
 }
 
@@ -72,16 +71,16 @@ deliver_queue() { # ()
 	for dir in $qdir/*; do
 		[ ! -d "$dir" ] && continue
 		sleep 5 # allow tunnel to close properly
-		send_mail $dir
+		send_mail "$dir"
 		undelivered=`expr $undelivered + $?`
 	done
 	return $undelivered
 }
 
-ME=`basename $0`
-case $ME in
+ME=`basename "$0"`
+case "$ME" in
 	sendmail)
-		queue_mail $@
+		queue_mail "$@" || return 1
 		deliver_queue &
 	;;
 	deliver)
-- 
cgit v1.2.3