download handling: adapt cycle check to check value

instead of whether the key exists in the schema instance, just in case somebody
wants to set "download => 0".

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
index 1a37b3ac7264c00c9702db99eb1e16b0c57b99ff..81e269fdcda04461fc8cd50aebb497b03bb48fed 100644 (file)
--- a/src/PVE/JSONSchema.pm
+++ b/src/PVE/JSONSchema.pm
@@ -1370,7 +1370,7 @@ sub validate {
     my $cycles = 0;
     # 'download' responses can contain a filehandle, don't cycle-check that as
     # it produces a warning
-    my $is_download = ref($instance) eq 'HASH' && exists($instance->{download});
+    my $is_download = ref($instance) eq 'HASH' && $instance->{download};
     find_cycle($instance, sub { $cycles = 1 }) if !$is_download;
     if ($cycles) {
        add_error($errors, undef, "data structure contains recursive cycles");