some ovfs to not declare 'rasd' as a default namespace
(in the top level Envelope element), but inline in each element
(e.g. <rasd:HostResource xmlns:rasd="foo">...</rasd:HostResource>)

this trips up our relative findvalue with

XPath error : Undefined namespace prefix

to avoid this, search in the global XPathContext (where we register
those namespaces ourselves) and give the item_node as context

this works in both cases

Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
---
 PVE/QemuServer/OVF.pm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/PVE/QemuServer/OVF.pm b/PVE/QemuServer/OVF.pm
index 7ee4dc8..536e0eb 100644
--- a/PVE/QemuServer/OVF.pm
+++ b/PVE/QemuServer/OVF.pm
@@ -155,7 +155,7 @@ sub parse_ovf {
 
        # from Item, find corresponding Disk node
        # here the dot means the search should start from the current element 
in dom
-       my $host_resource = $item_node->findvalue('./rasd:HostResource');
+       my $host_resource = $xpc->findvalue('rasd:HostResource', $item_node);
        my $disk_section_path;
        my $disk_id;
 
@@ -194,7 +194,7 @@ ovf:Disk[\@ovf:diskId='%s']/\@ovf:fileRef", $disk_id);
        print "file path: $filepath\n" if $debug;
 
        # from Item, find owning Controller type
-       my $controller_id = $item_node->findvalue('./rasd:Parent');
+       my $controller_id = $xpc->findvalue('rasd:Parent', $item_node);
        my $xpath_find_parent_type = 
sprintf("/ovf:Envelope/ovf:VirtualSystem/ovf:VirtualHardwareSection/\
 ovf:Item[rasd:InstanceID='%s']/rasd:ResourceType", $controller_id);
        my $controller_type = $xpc->findvalue($xpath_find_parent_type);
@@ -205,7 +205,7 @@ ovf:Item[rasd:InstanceID='%s']/rasd:ResourceType", 
$controller_id);
        print "owning controller type: $controller_type\n" if $debug;
 
        # extract corresponding Controller node details
-       my $adress_on_controller = 
$item_node->findvalue('./rasd:AddressOnParent');
+       my $adress_on_controller = $xpc->findvalue('rasd:AddressOnParent', 
$item_node);
        my $pve_disk_address = id_to_pve($controller_type) . 
$adress_on_controller;
 
        # resolve symlinks and relative path components
-- 
2.20.1


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

Reply via email to