Switch back to mkdir, and instead explicitly set permissions and ownership (even when not creating the dirs). Quote user a few places.

1 files changed, 6 insertions, 4 deletions

diff --git a/localdumpsql b/localdumpsql
index 5f1a25a..65833c1 100755
--- a/localdumpsql
+++ b/localdumpsql
@@ -3,7 +3,7 @@
 # /usr/local/sbin/localdumpsql
 # Copyright 2001-2003 Jonas Smedegaard <dr@jones.dk>
 #
-# $Id: localdumpsql,v 1.7 2003-05-22 11:36:25 jonas Exp $
+# $Id: localdumpsql,v 1.8 2003-05-22 11:43:28 jonas Exp $
 #
 # MySQL/PostgreSQL maintenance script
 #
@@ -87,12 +87,14 @@ for sqltype in $sqltypes; do
 			test -d "$home" || continue
 			targetdir="`getbackupdir_user \"$user\"`/$sqltype"
 			if [ ! -d "$targetdir" ]; then
-				install -d -oroot -g"$user" -m0750 "$targetdir"
+				mkdir -p "$targetdir"
 			fi
+			chmod -R u=rw,go=,u+X "$targetdir"
+			chown -R "$user" "$targetdir"
 			for db in `${sqltype}_db_user "$databases" "$user"`; do
 				targetfile="$targetdir/$stamp.$db.sql"
-				(set +e; ${sqltype}dump_user_db $user $db; set -e) > "$targetfile"
-				chown $user "$targetfile"
+				(set +e; ${sqltype}dump_user_db "$user" "$db"; set -e) > "$targetfile"
+				chown "$user" "$targetfile"
 			done
 		done
 	fi