Skip to content
Snippets Groups Projects
Commit 806d9765 authored by intrigeri's avatar intrigeri
Browse files

pgsql: Fixed shell expansion, thanks Thomas Kotzian (Closes: #363297)

parent 3313ebbd
No related branches found
No related tags found
No related merge requests found
...@@ -6,13 +6,14 @@ version 0.9.4 -- unreleased ...@@ -6,13 +6,14 @@ version 0.9.4 -- unreleased
Recursively ignore subdirs in /etc/backup.d (Closes: #361102) Recursively ignore subdirs in /etc/backup.d (Closes: #361102)
Added tar handler Added tar handler
mysql: mysql:
. Fixed improper use of $vuserhome (Debian: #351083) . Fixed improper use of $vuserhome (Closes: #351083)
. Fixed erroneous removal of tmpfile when it didn't exit . Fixed erroneous removal of tmpfile when it didn't exit
. Fixed inversed vsname emptiness check . Fixed inversed vsname emptiness check
. Fixed su quote usage to be more posixy . Fixed su quote usage to be more posixy
pgsql: pgsql:
. Fixed inversed vsname emptiness check . Fixed inversed vsname emptiness check
. Fixed su quote usage to be more posixy . Fixed su quote usage to be more posixy
. Fixed shell expansion, thanks Thomas Kotzian (Closes: #363297)
svn: svn:
. Fixed inversed vsname emptiness check . Fixed inversed vsname emptiness check
rdiff: rdiff:
......
...@@ -72,13 +72,13 @@ chmod 700 $vroot$backupdir ...@@ -72,13 +72,13 @@ chmod 700 $vroot$backupdir
# if $databases = all, use pg_dumpall # if $databases = all, use pg_dumpall
if [ "$databases" == "all" ]; then if [ "$databases" == "all" ]; then
if [ $usevserver = yes ]; then if [ $usevserver = yes ]; then
execstr='$VSERVER $vsname exec su - postgres -c \"$PGSQLDUMPALL > $backupdir/${vsname}.sql\"' execstr="$VSERVER $vsname exec su - postgres -c \"$PGSQLDUMPALL > $backupdir/${vsname}.sql\""
else else
execstr='su - postgres -c \"$PGSQLDUMPALL > $backupdir/${localhost}-all.sql\"' execstr="su - postgres -c \"$PGSQLDUMPALL > $backupdir/${localhost}-all.sql\""
fi fi
debug "$execstr" debug "$execstr"
if [ ! $test ]; then if [ ! $test ]; then
output=`$execstr 2>&1` output=`eval $execstr 2>&1`
code=$? code=$?
if [ "$code" == "0" ]; then if [ "$code" == "0" ]; then
debug $output debug $output
...@@ -94,13 +94,13 @@ else ...@@ -94,13 +94,13 @@ else
for db in $databases; do for db in $databases; do
if [ $usevserver = yes ] if [ $usevserver = yes ]
then then
execstr='$VSERVER $vsname exec su - postgres -c \"$PGSQLDUMP $db > $backupdir/${db}.sql\"' execstr="$VSERVER $vsname exec su - postgres -c \"$PGSQLDUMP $db > $backupdir/${db}.sql\""
else else
execstr='su - postgres -c \"$PGSQLDUMP $db > $backupdir/${db}.sql\"' execstr="su - postgres -c \"$PGSQLDUMP $db > $backupdir/${db}.sql\""
fi fi
debug "$execstr" debug "$execstr"
if [ ! $test ]; then if [ ! $test ]; then
output=`$execstr 2>&1` output=`eval $execstr 2>&1`
code=$? code=$?
if [ "$code" == "0" ]; then if [ "$code" == "0" ]; then
debug $output debug $output
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment