From 7fabddca0575489d51a606062ae2d5c0e6fa3671 Mon Sep 17 00:00:00 2001 From: Wolfgang Bumiller Date: Fri, 25 Sep 2015 09:41:40 +0200 Subject: [PATCH] CephTools: improve abs_path error handling verify_blockdev_path didn't check the result of abs_path causing commands like `pveceph createosd bad/path` to error with a meaningless "Use of uninitialized value" message. --- PVE/CephTools.pm | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/PVE/CephTools.pm b/PVE/CephTools.pm index 68bf1756..1c639e33 100644 --- a/PVE/CephTools.pm +++ b/PVE/CephTools.pm @@ -43,9 +43,11 @@ sub get_config { } sub verify_blockdev_path { - my ($path) = @_; + my ($rel_path) = @_; - $path = abs_path($path); + die "missing path" if !$rel_path; + my $path = abs_path($rel_path); + die "failed to get absolute path to $rel_path" if !$path; die "got unusual device path '$path'\n" if $path !~ m|^/dev/(.*)$|;