[pve-devel] [PATCH qemu-server] fix net regex in qm cleanup
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Sep 21 09:58:36 CEST 2022
I initially read this as like it was a recent (code) cleanup that broke it.
Fabian already applied it, so only two suggestion as for how to improve
this for a next similar time..
subject: "qm: fix net regex in 'cleanup' command"
Am 20/09/2022 um 16:50 schrieb Dominik Csapak:
> the '+' needs to be in the brackets, otherwise '$1' is not the right id
>
Add
Fixes: 3ea84aeb7c75d7b0bfbd9cb04440ad00072bfe0c
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> PVE/CLI/qm.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
> index 6a2e161..ca5d25f 100755
> --- a/PVE/CLI/qm.pm
> +++ b/PVE/CLI/qm.pm
> @@ -809,7 +809,7 @@ __PACKAGE__->register_method({
> # we have to cleanup the tap devices after a crash
>
> foreach my $opt (keys %$conf) {
> - next if $opt !~ m/^net(\d)+$/;
> + next if $opt !~ m/^net(\d+)$/;
> my $interface = $1;
> PVE::Network::tap_unplug("tap${vmid}i${interface}");
> }
More information about the pve-devel
mailing list