Hello community,

here is the log from the commit of package yast2-restore for openSUSE:Factory 
checked in at 2012-11-12 07:07:56
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-restore (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-restore.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-restore", Maintainer is "[email protected]"

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-restore/yast2-restore.changes      
2012-11-02 19:40:06.000000000 +0100
+++ /work/SRC/openSUSE:Factory/.yast2-restore.new/yast2-restore.changes 
2012-11-12 07:08:00.000000000 +0100
@@ -1,0 +2,7 @@
+Fri Nov  2 11:27:29 UTC 2012 - [email protected]
+
+- Escaping shell paths that might contain spaces or special
+  characters (BNC #776078)
+- 2.23.1
+
+-------------------------------------------------------------------

Old:
----
  yast2-restore-2.22.1.tar.bz2

New:
----
  yast2-restore-2.23.1.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-restore.spec ++++++
--- /var/tmp/diff_new_pack.FTaNeL/_old  2012-11-12 07:08:01.000000000 +0100
+++ /var/tmp/diff_new_pack.FTaNeL/_new  2012-11-12 07:08:01.000000000 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-restore
-Version:        2.22.1
+Version:        2.23.1
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-restore-2.22.1.tar.bz2 -> yast2-restore-2.23.1.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-restore-2.22.1/VERSION 
new/yast2-restore-2.23.1/VERSION
--- old/yast2-restore-2.22.1/VERSION    2012-11-01 13:36:32.000000000 +0100
+++ new/yast2-restore-2.23.1/VERSION    2012-11-09 13:38:53.000000000 +0100
@@ -1 +1 @@
-2.22.1
+2.23.1
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-restore-2.22.1/configure 
new/yast2-restore-2.23.1/configure
--- old/yast2-restore-2.22.1/configure  2012-11-01 13:19:23.000000000 +0100
+++ new/yast2-restore-2.23.1/configure  2012-11-09 13:39:07.000000000 +0100
@@ -1,6 +1,6 @@
 #! /bin/sh
 # Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.69 for yast2-restore 2.22.0.
+# Generated by GNU Autoconf 2.69 for yast2-restore 2.23.1.
 #
 # Report bugs to <http://bugs.opensuse.org/>.
 #
@@ -579,8 +579,8 @@
 # Identity of this package.
 PACKAGE_NAME='yast2-restore'
 PACKAGE_TARNAME='yast2-restore'
-PACKAGE_VERSION='2.22.0'
-PACKAGE_STRING='yast2-restore 2.22.0'
+PACKAGE_VERSION='2.23.1'
+PACKAGE_STRING='yast2-restore 2.23.1'
 PACKAGE_BUGREPORT='http://bugs.opensuse.org/'
 PACKAGE_URL=''
 
@@ -1247,7 +1247,7 @@
   # Omit some internal or obsolete options to make the list less imposing.
   # This message is too long to be a string in the A/UX 3.1 sh.
   cat <<_ACEOF
-\`configure' configures yast2-restore 2.22.0 to adapt to many kinds of systems.
+\`configure' configures yast2-restore 2.23.1 to adapt to many kinds of systems.
 
 Usage: $0 [OPTION]... [VAR=VALUE]...
 
@@ -1318,7 +1318,7 @@
 
 if test -n "$ac_init_help"; then
   case $ac_init_help in
-     short | recursive ) echo "Configuration of yast2-restore 2.22.0:";;
+     short | recursive ) echo "Configuration of yast2-restore 2.23.1:";;
    esac
   cat <<\_ACEOF
 
@@ -1398,7 +1398,7 @@
 test -n "$ac_init_help" && exit $ac_status
 if $ac_init_version; then
   cat <<\_ACEOF
-yast2-restore configure 2.22.0
+yast2-restore configure 2.23.1
 generated by GNU Autoconf 2.69
 
 Copyright (C) 2012 Free Software Foundation, Inc.
@@ -1415,7 +1415,7 @@
 This file contains any messages produced by compilers while
 running configure, to aid debugging if configure makes a mistake.
 
-It was created by yast2-restore $as_me 2.22.0, which was
+It was created by yast2-restore $as_me 2.23.1, which was
 generated by GNU Autoconf 2.69.  Invocation command line was
 
   $ $0 $@
@@ -2354,7 +2354,7 @@
 
 # Define the identity of the package.
  PACKAGE='yast2-restore'
- VERSION='2.22.0'
+ VERSION='2.23.1'
 
 
 cat >>confdefs.h <<_ACEOF
@@ -2477,7 +2477,7 @@
 
 
 
-VERSION="2.22.0"
+VERSION="2.23.1"
 RPMNAME="yast2-restore"
 MAINTAINER="Lukas Ocilka <[email protected]>"
 
@@ -3403,7 +3403,7 @@
 # report actual input values of CONFIG_FILES etc. instead of their
 # values after options handling.
 ac_log="
-This file was extended by yast2-restore $as_me 2.22.0, which was
+This file was extended by yast2-restore $as_me 2.23.1, which was
 generated by GNU Autoconf 2.69.  Invocation command line was
 
   CONFIG_FILES    = $CONFIG_FILES
@@ -3456,7 +3456,7 @@
 cat >>$CONFIG_STATUS <<_ACEOF || ac_write_fail=1
 ac_cs_config="`$as_echo "$ac_configure_args" | sed 's/^ //; 
s/[\\""\`\$]/\\\\&/g'`"
 ac_cs_version="\\
-yast2-restore config.status 2.22.0
+yast2-restore config.status 2.23.1
 configured by $0, generated by GNU Autoconf 2.69,
   with options \\"\$ac_cs_config\\"
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-restore-2.22.1/configure.in 
new/yast2-restore-2.23.1/configure.in
--- old/yast2-restore-2.22.1/configure.in       2012-11-01 13:19:21.000000000 
+0100
+++ new/yast2-restore-2.23.1/configure.in       2012-11-09 13:39:05.000000000 
+0100
@@ -3,7 +3,7 @@
 dnl -- This file is generated by y2autoconf 2.23.0 - DO NOT EDIT! --
 dnl    (edit configure.in.in instead)
 
-AC_INIT(yast2-restore, 2.22.0, http://bugs.opensuse.org/, yast2-restore)
+AC_INIT(yast2-restore, 2.23.1, http://bugs.opensuse.org/, yast2-restore)
 dnl Check for presence of file 'RPMNAME'
 AC_CONFIG_SRCDIR([RPMNAME])
 
@@ -18,7 +18,7 @@
 AM_INIT_AUTOMAKE(tar-ustar -Wno-portability)
 
 dnl Important YaST2 variables
-VERSION="2.22.0"
+VERSION="2.23.1"
 RPMNAME="yast2-restore"
 MAINTAINER="Lukas Ocilka <[email protected]>"
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-restore-2.22.1/src/Restore.ycp 
new/yast2-restore-2.23.1/src/Restore.ycp
--- old/yast2-restore-2.22.1/src/Restore.ycp    2012-11-01 13:16:11.000000000 
+0100
+++ new/yast2-restore-2.23.1/src/Restore.ycp    2012-11-09 13:38:53.000000000 
+0100
@@ -694,7 +694,7 @@
        boolean lastvolume = true;
 
        // test if archive is multi volume - use -v parameter to get file 
descriptions
-       map detailresult = (map) SCR::Execute(.target.bash_output, "/bin/tar -v 
-t -f " + filename);
+       map detailresult = (map) SCR::Execute(.target.bash_output, "/bin/tar -v 
-t -f '" + String::Quote(filename) + "'");
 
        list<string> stdout = splitstring(detailresult["stdout"]:"", "\n");
        string firstline = stdout[0]:"";
@@ -723,7 +723,7 @@
        string tmpfile = tempdir + "/" + sformat("%1", num) + ".tar";
 
        // copy multi volume part to temp directory
-       integer exit = (integer) SCR::Execute(.target.bash, "/bin/cp " + 
filename + " " + tmpfile);
+       integer exit = (integer) SCR::Execute(.target.bash, "/bin/cp '" + 
String::Quote(filename) + "' '" + String::Quote(tmpfile) + "'");
 
        if (exit == 0)
        {
@@ -909,7 +909,7 @@
        y2milestone("mountpoint: %1", mountpoint);
 
        // get archive contents
-       map result = (map) SCR::Execute(.target.bash_output, "/bin/tar -t -f " 
+ filename);
+       map result = (map) SCR::Execute(.target.bash_output, "/bin/tar -t -f '" 
+ String::Quote(filename) + "'");
 
        // check tar exit value
        if (result["exit"]:-1 != 0)
@@ -987,7 +987,7 @@
        infofiles = infofiles + (compressed_packages_info ? 
"info/packages_info.gz" : "info/packages_info");
 
        // unpack info files
-       result = (map) SCR::Execute(.target.bash_output, "/bin/tar -C '" + 
String::Quote (tempdir) + "' -x -f " + filename + " " + infofiles + " 2> 
/dev/null");
+       result = (map) SCR::Execute(.target.bash_output, "/bin/tar -C '" + 
String::Quote (tempdir) + "' -x -f '" + String::Quote(filename) + "' " + 
infofiles + " 2> /dev/null");
 
        date = (string) SCR::Read(.target.string, tempdir + "/info/date");
        comment = (string) SCR::Read(.target.string, tempdir + "/info/comment");
@@ -1491,18 +1491,18 @@
                        string param = " ";
 
                        foreach(string f, volumeparts, ``{
-                               param = param + "-f " + f + " ";
+                               param = param + "-f '" + String::Quote(f) + "' 
";
                            }
                        );
 
-                       string command = locale_modifications + "echo q | 
/bin/tar -C " + tempdir + " -x -M " + param + fileinarchive + " 2> /dev/null";
+                       string command = locale_modifications + "echo q | 
/bin/tar -C " + tempdir + " -x -M " + param + "'" + 
String::Quote(fileinarchive) + "' 2> /dev/null";
                        y2milestone ("Running command: %1", command);
 
                        started = (boolean) SCR::Execute(.background.run, 
command);
                    }
                    else
                    {
-                       string command = locale_modifications + "/bin/tar -C " 
+ tempdir + " -x -f " + filename + " " + fileinarchive;
+                       string command = locale_modifications + "/bin/tar -C " 
+ tempdir + " -x -f '" + String::Quote(filename) + "' '" + 
String::Quote(fileinarchive) + "'";
                        y2milestone ("Running command: %1", command);
 
                        started  = (boolean) SCR::Execute(.background.run, 
command);

-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to