From e94fffc1c1c822b16b82dcf7dab8d82ed2625d3d Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Sun, 14 Mar 1999 16:00:55 +0000
Subject: [PATCH] This patch improves the destroyuser script's behaviour in
 checking whether the user is allowed to detroy users.  It also allows for a
 possible incorrect empty reply to a yes/no question.

Oliver Elphick
---
 src/bin/destroyuser/destroyuser.sh | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/bin/destroyuser/destroyuser.sh b/src/bin/destroyuser/destroyuser.sh
index aa3e13a94f..135fff181f 100644
--- a/src/bin/destroyuser/destroyuser.sh
+++ b/src/bin/destroyuser/destroyuser.sh
@@ -8,7 +8,7 @@
 #
 #
 # IDENTIFICATION
-#    $Header: /cvsroot/pgsql/src/bin/destroyuser/Attic/destroyuser.sh,v 1.10 1998/09/01 15:57:00 thomas Exp $
+#    $Header: /cvsroot/pgsql/src/bin/destroyuser/Attic/destroyuser.sh,v 1.11 1999/03/14 16:00:55 momjian Exp $
 #
 # Note - this should NOT be setuid.
 #
@@ -79,9 +79,10 @@ then
     exit 1
 fi
 
-if [ $ADDUSER != "t" ]
+if [ x$ADDUSER != xt ]
 then
     echo "$CMDNAME: $USER cannot delete users."
+    exit 1
 fi
 
 #
@@ -152,7 +153,7 @@ then
 #
 
     yn=f
-    while [ $yn != y -a $yn != n ]
+    while [ "$yn" != y -a "$yn" != n ]
     do
         echo PG_OPT_DASH_N_PARAM "Deleting user $DELUSER will destroy them. Continue (y/n)? PG_OPT_BACKSLASH_C_PARAM"
         read yn
-- 
GitLab