ambiser
/
btrbk
mirror of https://github.com/digint/btrbk
9
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

btrbk: remove double-slash from file arguments 

Sanitize file (or subvolume path) arguments in safe_cmd, effectively
removing leading double slash.

Files originating from "volume /" can be assembled as "//some/subvol",
which is useful internally but undesired as command arguments, as
ancient systems might interpret leading double slash "//" in a special
way.

Posix states:

> A pathname that begins with two successive slashes may be
> interpreted in an implementation-defined manner, although more than
> two leading slashes shall be treated as a single slash.
pull/542/head
Axel Burri 2022-12-02 22:32:17 +01:00
parent b9c5e3fc29
commit 799d235218
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
btrbk
View File

@ -806,10 +806,12 @@ sub _safe_cmd($;$)
$_ = $_->{unsafe}; $_ = $_->{unsafe};
die "cannot quote leading dash for command: $_" if(/^-/); die "cannot quote leading dash for command: $_" if(/^-/);
# NOTE: all files must be absolute # NOTE: all files must be absolute
if($offending) { my $file = check_file($_, { absolute => 1 }, sanitize => 1 );
push @$offending, $_ unless(defined(check_file($_, { absolute => 1 }))); unless(defined($file)) {
die "uncaught unsafe file: $_" unless($offending);
push @$offending, $_;
} }
$_ = $prefix . quoteshell($_) . $postfix; $_ = $prefix . quoteshell($file // $_) . $postfix;
} }
$_ $_
} @$aref; } @$aref;