[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