]> git.proxmox.com Git - pve-container.git/commitdiff
lxc: fix perl-prototyep of mountpoint_hotplug
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 21 Oct 2022 13:20:34 +0000 (15:20 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 21 Oct 2022 13:23:24 +0000 (15:23 +0200)
The original commit b2de4c048ee50094593f4f8ffd18b6c346f7157a
copy-pasta'd the wrong prototype, missing two scalar $ arguments.

Until recently perl did not care as those things are only checked
_somewhat_ on "compile" (module load) times, and the one (single?)
call site in PVE::LXC::Config missed the `use PVE::LXC` statement,
and so the module-load did not see the wrong prototype and thus did
not cared, on runtime all is different anyway (what a mess).

The recent commit 11066f6bfdca5225a6f872d5664e6637ccb58dd6 added that
use statement and made package compilation implode, almost like
spooky actions in the time-space distance...

Fixes: b2de4c048ee50094593f4f8ffd18b6c346f7157a
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/PVE/LXC.pm

index fe630875470ef533601d1ea7fee887f01150396b..333286a25a7e58ac4d7aaaf66e56bca32a9c4ed2 100644 (file)
@@ -1814,7 +1814,7 @@ sub __mountpoint_mount {
     die "unsupported storage";
 }
 
-sub mountpoint_hotplug($$$) {
+sub mountpoint_hotplug :prototype($$$$$) {
     my ($vmid, $conf, $opt, $mp, $storage_cfg) = @_;
 
     my (undef, $rootuid, $rootgid) = PVE::LXC::parse_id_maps($conf);