diff -up libvirt-php-0.4.5/src/libvirt-php.c.php54 libvirt-php-0.4.5/src/libvirt-php.c --- libvirt-php-0.4.5/src/libvirt-php.c.php54 2011-11-27 16:49:11.201682937 +0100 +++ libvirt-php-0.4.5/src/libvirt-php.c 2011-11-27 17:05:24.092827417 +0100 @@ -16,6 +16,11 @@ #include "libvirt-php.h" +// From vncfunc.c +int vnc_get_dimensions(char *server, char *port, int *width, int *height); +// From sockets.c +int connect_socket(char *server, char *port, int keepalive, int nodelay, int allow_server_override); + #ifdef DEBUG_CORE #define DPRINTF(fmt, ...) \ if (LIBVIRT_G(debug)) \ @@ -50,7 +55,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_libvirt_c ZEND_ARG_INFO(0, readonly) ZEND_END_ARG_INFO() -static function_entry libvirt_functions[] = { +static zend_function_entry libvirt_functions[] = { /* Common functions */ PHP_FE(libvirt_get_last_error,NULL) /* Connect functions */ @@ -775,7 +780,7 @@ unsigned long long size_def_to_mbytes(ch return 0; unit = arg[strlen(arg)-1]; - switch (arg[strlen(arg)-1]) { + switch (unit) { case 'G': multiplicator = 1 << 10; break; @@ -1419,7 +1424,7 @@ PHP_FUNCTION(libvirt_connect) if (zend_hash_get_current_key_ex(arr_hash, &key, &key_len, &index, 0, &pointer) == HASH_KEY_IS_STRING) { PHPWRITE(key, key_len); } else { - DPRINTF("%s: credentials index %d\n", PHPFUNC, index); + DPRINTF("%s: credentials index %d\n", PHPFUNC, (int)index); creds[j].type=index; creds[j].result=emalloc( Z_STRLEN_PP(data) + 1 ); memset(creds[j].result, 0, Z_STRLEN_PP(data) + 1); @@ -1623,7 +1628,7 @@ PHP_FUNCTION(libvirt_image_create) zval *zconn; char msg[1024]; char cmd[4096] = { 0 }; - char *path; + char *path = NULL; char fpath[4096] = { 0 }; char *image = NULL; int image_len; @@ -1965,7 +1970,7 @@ void dec_to_bin(unsigned long long decim int k = 0, n = 0; int neg_flag = 0; int remain; - int old_decimal; + // int old_decimal; char temp[128] = { 0 }; if (decimal < 0) @@ -1975,7 +1980,7 @@ void dec_to_bin(unsigned long long decim } do { - old_decimal = decimal; + // old_decimal = decimal; remain = decimal % 2; decimal = decimal / 2; temp[k++] = remain + '0'; @@ -2050,7 +2055,7 @@ long get_next_free_numeric_value(virDoma int retval = -1; HashTable *arr_hash; HashPosition pointer; - int array_count; + // int array_count; zval **data; char *key; unsigned int key_len; @@ -2063,7 +2068,7 @@ long get_next_free_numeric_value(virDoma free( get_string_from_xpath(xml, xpath, &output, &retval) ); arr_hash = Z_ARRVAL_P(output); - array_count = zend_hash_num_elements(arr_hash); + // array_count = zend_hash_num_elements(arr_hash); for (zend_hash_internal_pointer_reset_ex(arr_hash, &pointer); zend_hash_get_current_data_ex(arr_hash, (void**) &data, &pointer) == SUCCESS; zend_hash_move_forward_ex(arr_hash, &pointer)) { @@ -2188,7 +2193,7 @@ char *connection_get_arch(virConnectPtr int retval = -1; char *tmp = NULL; char *caps = NULL; - char xpath[1024] = { 0 }; + // char xpath[1024] = { 0 }; caps = virConnectGetCapabilities(conn); if (caps == NULL) @@ -2219,7 +2224,7 @@ char *generate_uuid_any() int i; char a[37] = { 0 }; char hexa[] = "0123456789abcdef"; - virDomainPtr domain=NULL; + // virDomainPtr domain=NULL; srand(time(NULL)); for (i = 0; i < 36; i++) { @@ -2304,7 +2309,7 @@ char *get_disk_xml(unsigned long long si ret = WEXITSTATUS(system(cmd)); DPRINTF("%s: Command '%s' finished with error code %d\n", __FUNCTION__, cmd, ret); if (ret != 0) { - DPRINTF("%s: File creation failed\n"); + DPRINTF("%s: File creation failed\n", path); return NULL; } @@ -2387,7 +2392,7 @@ char *installation_get_xml(int step, vir char features[128] = { 0 }; char *tmp = NULL; char type[64] = { 0 }; - virDomainPtr domain=NULL; + // virDomainPtr domain=NULL; if (conn == NULL) { DPRINTF("%s: Invalid libvirt connection pointer\n", __FUNCTION__); @@ -2978,7 +2983,7 @@ PHP_FUNCTION(libvirt_domain_send_keys) GET_DOMAIN_FROM_ARGS("rss",&zdomain, &hostname, &hostname_len, &keys, &keys_len); - DPRINTF("%s: Sending %d VNC keys to %s...\n", PHPFUNC, strlen(keys), hostname); + DPRINTF("%s: Sending %d VNC keys to %s...\n", PHPFUNC, (int)strlen(keys), hostname); xml=virDomainGetXMLDesc(domain->domain, 0); if (xml==NULL) { @@ -2992,7 +2997,7 @@ PHP_FUNCTION(libvirt_domain_send_keys) RETURN_FALSE; } - DPRINTF("%s: About to send string '%s' (%d keys) to %s:%s\n", PHPFUNC, keys, strlen(keys), hostname, tmp); + DPRINTF("%s: About to send string '%s' (%d keys) to %s:%s\n", PHPFUNC, keys, (int)strlen(keys), hostname, tmp); ret = vnc_send_keys(hostname, tmp, keys); DPRINTF("%s: Sequence sending result is %d\n", PHPFUNC, ret); @@ -3214,15 +3219,15 @@ PHP_FUNCTION(libvirt_connect_get_emulato void parse_array(zval *arr, tVMDisk *disk, tVMNetwork *network) { HashTable *arr_hash; - int array_count; - zval **zvalue, **data; + // int array_count; + zval **data; // removed **zvalue HashPosition pointer; char *key; unsigned int key_len; unsigned long index; arr_hash = Z_ARRVAL_P(arr); - array_count = zend_hash_num_elements(arr_hash); + //array_count = zend_hash_num_elements(arr_hash); if (disk != NULL) memset(disk, 0, sizeof(tVMDisk)); @@ -3297,7 +3302,7 @@ PHP_FUNCTION(libvirt_domain_new) char *tmp; char *name; char name_len=0; - char *emulator; + // char *emulator; char *iso_image = NULL; int iso_image_len; int vcpus = -1; @@ -3308,7 +3313,7 @@ PHP_FUNCTION(libvirt_domain_new) int maxmemMB = -1; HashTable *arr_hash; int numDisks, numNets, i; - zval **zvalue, **data; + zval **data; // removed **zvalue HashPosition pointer; char vncl[2048] = { 0 }; char tmpname[1024] = { 0 }; @@ -3677,7 +3682,7 @@ PHP_FUNCTION(libvirt_domain_change_memor long allocMem = 0; long allocMax = 0; int retval = -1; - int pos = -1; + // int pos = -1; int len = 0; php_libvirt_domain *res_domain = NULL; php_libvirt_connection *conn = NULL; @@ -3703,7 +3708,7 @@ PHP_FUNCTION(libvirt_domain_change_memor snprintf(new, sizeof(new), " %d\n %d\n", allocMax, allocMem); tmpA = strstr(xml, ""); tmp1 = strstr(xml, "") + strlen(""); - pos = strlen(xml) - strlen(tmp1); + // pos = strlen(xml) - strlen(tmp1); len = strlen(xml) - strlen(tmpA); tmp2 = emalloc( ( len + 1 )* sizeof(char) ); @@ -3770,7 +3775,7 @@ PHP_FUNCTION(libvirt_domain_change_boot_ char *second = NULL; int second_len; int retval = -1; - int pos = -1; + // int pos = -1; int len = 0; php_libvirt_domain *res_domain = NULL; php_libvirt_connection *conn = NULL; @@ -3793,7 +3798,7 @@ PHP_FUNCTION(libvirt_domain_change_boot_ tmpA = strstr(xml, "") + strlen(""); tmp1 = strstr(xml, ""); - pos = strlen(xml) - strlen(tmp1); + // pos = strlen(xml) - strlen(tmp1); len = strlen(xml) - strlen(tmpA); tmp2 = emalloc( ( len + 1 )* sizeof(char) );