[pve-devel] applied: [PATCH qemu-server] Fix guest agent shutdown without timeout
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Jun 25 16:52:41 CEST 2019
On 6/25/19 4:44 PM, Stefan Reiter wrote:
> Regression from change allowing timeouts to be set, now shutting down
> also works without timeouts again (previously qmp failed because of
> the unknown "timeout" parameter passed to it).
>
> We always delete the timeout value from the arguments, regardless of
> truthiness. "delete" returns the deleted element, deleting a
> non-existant hash entry returns undef, which is fine after this point:
>
> "deleting non-existent elements returns the undefined value in their
> corresponding positions."
> - https://perldoc.perl.org/functions/delete.html
>
> Signed-off-by: Stefan Reiter <s.reiter at proxmox.com>
> ---
> PVE/QemuServer.pm | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
> index bf71210..fbfc3fb 100644
> --- a/PVE/QemuServer.pm
> +++ b/PVE/QemuServer.pm
> @@ -5485,9 +5485,8 @@ sub vm_qmp_command {
> my $res;
>
> my $timeout;
> - if ($cmd->{arguments} && $cmd->{arguments}->{timeout}) {
> - $timeout = $cmd->{arguments}->{timeout};
> - delete $cmd->{arguments}->{timeout};
> + if ($cmd->{arguments}) {
> + $timeout = delete $cmd->{arguments}->{timeout};
> }
>
> eval {
>
applied, thanks!
More information about the pve-devel
mailing list