Currently libvirt_connect fails if libvirt has no Xen support.
This is because virGetVersion checks for Xen if typeVer!=NULL.
---
 src/libvirt.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/libvirt.c b/src/libvirt.c
index ce39a28..9a1b51a 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -173,13 +173,12 @@ PHP_RSHUTDOWN_FUNCTION(libvirt)
 PHP_MINFO_FUNCTION(libvirt)
 {
        unsigned long libVer;
-       unsigned long typeVer;
        char *version;
        php_info_print_table_start();
        php_info_print_table_row(2, "Libvirt support", "enabled");
        php_info_print_table_row(2, "Extension version", 
PHP_LIBVIRT_WORLD_VERSION);
 
-       if (virGetVersion(&libVer,NULL,&typeVer)== 0)
+       if (virGetVersion(&libVer,NULL,NULL)== 0)
        {
                version=emalloc(100);
                snprintf(version, 100, "%i.%i.%i", (long)((libVer/1000000) % 
1000),(long)((libVer/1000) % 1000),(long)(libVer % 1000));
@@ -590,13 +589,12 @@ PHP_FUNCTION(libvirt_connect)
        unsigned long index;
 
        unsigned long libVer;
-       unsigned long typeVer;
 
        if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|sba", 
&url,&url_len,&readonly,&zcreds) == FAILURE) {
                RETURN_FALSE;
        }
 
-       if (virGetVersion(&libVer,NULL,&typeVer)!= 0)
+       if (virGetVersion(&libVer,NULL,NULL)!= 0)
                RETURN_FALSE;
 
        if (libVer<6002)
-- 
1.7.4.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to