- added missing shell quoting

diff --git a/package/snapper.changes b/package/snapper.changes
index fd05316035ede74c01e7a31885c069c7f82e1bc0..01ad5097e5a4f6ddcc8ab289cbafc7219735696f 100644 (file)
--- a/package/snapper.changes
+++ b/package/snapper.changes
@@ -1,3 +1,8 @@
+-------------------------------------------------------------------
+Sat Jul 30 16:15:42 CEST 2011 - aschnell@suse.de
+
+- added missing shell quoting
+
 -------------------------------------------------------------------
 Tue Jun 28 11:46:28 CEST 2011 - aschnell@suse.de
 

diff --git a/snapper/Filesystem.cc b/snapper/Filesystem.cc
index 1086cfe67f19c53905b875cf9f96f354d3baf66e..b374df1c178dcd7a4287cc3fdc547dd91e29d68a 100644 (file)
--- a/snapper/Filesystem.cc
+++ b/snapper/Filesystem.cc
@@ -36,7 +36,7 @@ namespace snapper
     void
     Btrfs::addConfig() const
     {
-       SystemCmd cmd2(BTRFSBIN " subvolume create " + subvolume + "/.snapshots");
+       SystemCmd cmd2(BTRFSBIN " subvolume create " + quote(subvolume + "/.snapshots"));
        if (cmd2.retcode() != 0)
        {
            throw AddConfigFailedException("creating btrfs snapshot failed");
@@ -61,7 +61,8 @@ namespace snapper
     void
     Btrfs::createFilesystemSnapshot(unsigned int num) const
     {
-       SystemCmd cmd(BTRFSBIN " subvolume snapshot " + subvolume + " " + snapshotDir(num));
+       SystemCmd cmd(BTRFSBIN " subvolume snapshot " + quote(subvolume) + " " +
+                     quote(snapshotDir(num)));
        if (cmd.retcode() != 0)
            throw CreateSnapshotFailedException();
     }
@@ -70,7 +71,7 @@ namespace snapper
     void
     Btrfs::deleteFilesystemSnapshot(unsigned int num) const
     {
-       SystemCmd cmd(BTRFSBIN " subvolume delete " + snapshotDir(num));
+       SystemCmd cmd(BTRFSBIN " subvolume delete " + quote(snapshotDir(num)));
        if (cmd.retcode() != 0)
            throw DeleteSnapshotFailedException();
     }
@@ -125,11 +126,11 @@ namespace snapper
     void
     Ext4::createFilesystemSnapshot(unsigned int num) const
     {
-       SystemCmd cmd1(TOUCHBIN " " + snapshotFile(num));
+       SystemCmd cmd1(TOUCHBIN " " + quote(snapshotFile(num)));
        if (cmd1.retcode() != 0)
            throw CreateSnapshotFailedException();
 
-       SystemCmd cmd2(CHSNAPBIN " +S " + snapshotFile(num));
+       SystemCmd cmd2(CHSNAPBIN " +S " + quote(snapshotFile(num)));
        if (cmd2.retcode() != 0)
            throw CreateSnapshotFailedException();
     }
@@ -138,7 +139,7 @@ namespace snapper
     void
     Ext4::deleteFilesystemSnapshot(unsigned int num) const
     {
-       SystemCmd cmd(CHSNAPBIN " -S " + snapshotFile(num));
+       SystemCmd cmd(CHSNAPBIN " -S " + quote(snapshotFile(num)));
        if (cmd.retcode() != 0)
             throw DeleteSnapshotFailedException();
 
@@ -149,14 +150,14 @@ namespace snapper
     void
     Ext4::mountFilesystemSnapshot(unsigned int num) const
     {
-       SystemCmd cmd1(CHSNAPBIN " +n " + snapshotFile(num));
+       SystemCmd cmd1(CHSNAPBIN " +n " + quote(snapshotFile(num)));
        if (cmd1.retcode() != 0)
            throw MountSnapshotFailedException();
 
        mkdir(snapshotDir(num).c_str(), 0755);
 
-       SystemCmd cmd2(MOUNTBIN " -t ext4 -r -o loop,noload " + snapshotFile(num) +
-                      " " + snapshotDir(num));
+       SystemCmd cmd2(MOUNTBIN " -t ext4 -r -o loop,noload " + quote(snapshotFile(num)) +
+                      " " + quote(snapshotDir(num)));
        if (cmd2.retcode() != 0)
            throw MountSnapshotFailedException();
     }