On 3/3/20 10:47 AM, Dominik Csapak wrote:
> by using the new spice_port_range sub from PVE::Tools
> 
> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  PVE/APIServer/AnyEvent.pm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/PVE/APIServer/AnyEvent.pm b/PVE/APIServer/AnyEvent.pm
> index 3ce948f..664afbd 100644
> --- a/PVE/APIServer/AnyEvent.pm
> +++ b/PVE/APIServer/AnyEvent.pm
> @@ -808,7 +808,8 @@ sub handle_spice_proxy_request {
>  
>      eval {
>  
> -        die "Port $spiceport is not allowed" if ($spiceport < 61000 || 
> $spiceport > 61099);
> +     my ($minport, $maxport) = PVE::Tools::spice_port_range();
> +        die "Port $spiceport is not allowed" if ($spiceport < $minport || 
> $spiceport > $maxport);
>  
>       my $clientip = $reqstate->{peer_host};
>       my $r = $reqstate->{request};
> 

applied, with indentation/error message followup and versioned dependency bump
for pve-common in d/control. Thanks!

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to