Package: vbackup
Version: 1.0.1-1
Severity: important

The mbr script misses on / characters in device paths (like, when the device
is /dev/cciss/c0d0 in an HP array).  It inserts the part after /dev/
verbatim into its temp filenames.

A fix in the next stretch point release would be most welcome.

Find attached a patch with a simple fix.

-- System Information:
Debian Release: 9.3
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-5-amd64 (SMP w/8 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), 
LANGUAGE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

-- no debconf information
--- mbr.orig    2018-02-16 21:47:46.415903625 +0100
+++ mbr 2018-02-16 21:49:49.710561189 +0100
@@ -133,6 +133,7 @@
 
        for p in $DISKS ; do
                h_msg 12 "Backing up $p"
+               mkdir -p "$TDIR2/$p"
                dd if=/dev/$p "of=$TDIR2/$p.mbr" bs=512 count=1 >/dev/null 2>&1
                [ -z "$FDISK" ] || $FDISK -l /dev/$p > "$TDIR2/$p.fdisk-l" \
                        2> >(h_filter 6 >&2)

Reply via email to