[libvirt-php PATCH 21/35] Fix max_connections management

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Remi Collet <fedora@xxxxxxxxxxxxxxxxx>

- %p doesn't work with PHP snprintf
- store max_connections_ini as long instead of string
- store mem as pointer instead of integer
- convert test-conn-limit to phpt
---
 src/libvirt-php.c | 33 ++++++++++++---------------------
 src/libvirt-php.h |  4 ++--
 2 files changed, 14 insertions(+), 23 deletions(-)

diff --git a/src/libvirt-php.c b/src/libvirt-php.c
index ee79f0e..af80f06 100644
--- a/src/libvirt-php.c
+++ b/src/libvirt-php.c
@@ -682,7 +682,7 @@ PHP_INI_BEGIN()
 STD_PHP_INI_ENTRY("libvirt.longlong_to_string", "1", PHP_INI_ALL, OnUpdateBool, longlong_to_string_ini, zend_libvirt_globals, libvirt_globals)
 STD_PHP_INI_ENTRY("libvirt.iso_path", "/var/lib/libvirt/images/iso", PHP_INI_ALL, OnUpdateString, iso_path_ini, zend_libvirt_globals, libvirt_globals)
 STD_PHP_INI_ENTRY("libvirt.image_path", "/var/lib/libvirt/images", PHP_INI_ALL, OnUpdateString, image_path_ini, zend_libvirt_globals, libvirt_globals)
-STD_PHP_INI_ENTRY("libvirt.max_connections", "5", PHP_INI_ALL, OnUpdateString, max_connections_ini, zend_libvirt_globals, libvirt_globals)
+STD_PHP_INI_ENTRY("libvirt.max_connections", "5", PHP_INI_ALL, OnUpdateLong, max_connections_ini, zend_libvirt_globals, libvirt_globals)
 PHP_INI_END()
 
 void change_debug(int val TSRMLS_DC)
@@ -697,7 +697,7 @@ static void php_libvirt_init_globals(zend_libvirt_globals *libvirt_globals TSRML
     libvirt_globals->longlong_to_string_ini = 1;
     libvirt_globals->iso_path_ini = "/var/lib/libvirt/images/iso";
     libvirt_globals->image_path_ini = "/var/lib/libvirt/images";
-    libvirt_globals->max_connections_ini = "5";
+    libvirt_globals->max_connections_ini = 5;
     libvirt_globals->binding_resources_count = 0;
     libvirt_globals->binding_resources = NULL;
 #ifdef DEBUG_SUPPORT
@@ -877,18 +877,13 @@ void free_tokens(tTokenizer t)
  *                          @inc [int/bool]: Increment the counter (1 = add memory location) or decrement the counter (0 = remove memory location) from entries.
  * Returns:                 0 on success, -errno otherwise
  */
-int resource_change_counter(int type, virConnectPtr conn, void *memp, int inc TSRMLS_DC)
+int resource_change_counter(int type, virConnectPtr conn, void *mem, int inc TSRMLS_DC)
 {
     int i;
     int pos = -1;
     int binding_resources_count;
-    char tmp[64] = { 0 };
-    arch_uint mem = 0;
     resource_info *binding_resources = NULL;
 
-    snprintf(tmp, sizeof(tmp), "%p", memp);
-    sscanf(tmp, "%"UINTx, &mem);
-
     binding_resources_count = LIBVIRT_G(binding_resources_count);
     binding_resources = LIBVIRT_G(binding_resources);
 
@@ -1011,7 +1006,8 @@ PHP_MINFO_FUNCTION(libvirt)
         php_info_print_table_row(2, "Libvirt version", version);
     }
 
-    php_info_print_table_row(2, "Max. connections", LIBVIRT_G(max_connections_ini));
+    snprintf(path, sizeof(path), "%lu", (unsigned long)LIBVIRT_G(max_connections_ini));
+    php_info_print_table_row(2, "Max. connections", path);
 
     if (!access(LIBVIRT_G(iso_path_ini), F_OK) == 0)
         snprintf(path, sizeof(path), "%s - path is invalid. To set the valid path modify the libvirt.iso_path in your php.ini configuration!",
@@ -1254,16 +1250,11 @@ void free_resources_on_connection(virConnectPtr conn TSRMLS_DC)
  *                          @memp [pointer]: pointer to the memory
  * Returns:                 1 if resource is allocated, 0 otherwise
  */
-int check_resource_allocation(virConnectPtr conn, int type, void *memp TSRMLS_DC)
+int check_resource_allocation(virConnectPtr conn, int type, void *mem TSRMLS_DC)
 {
     int binding_resources_count = 0;
     resource_info *binding_resources = NULL;
     int i, allocated = 0;
-    char tmp[64] = { 0 };
-    arch_uint mem = 0;
-
-    snprintf(tmp, sizeof(tmp), "%p", memp);
-    sscanf(tmp, "%"UINTx, &mem);
 
     binding_resources_count = LIBVIRT_G(binding_resources_count);
     binding_resources = LIBVIRT_G(binding_resources);
@@ -2120,8 +2111,8 @@ PHP_FUNCTION(libvirt_connect)
         RETURN_FALSE;
     }
 
-    if ((count_resources(INT_RESOURCE_CONNECTION TSRMLS_CC) + 1) > atoi(LIBVIRT_G(max_connections_ini))) {
-        DPRINTF("%s: maximum number of connections allowed exceeded (max %s)\n",PHPFUNC, LIBVIRT_G(max_connections_ini));
+    if ((count_resources(INT_RESOURCE_CONNECTION TSRMLS_CC) + 1) > LIBVIRT_G(max_connections_ini)) {
+        DPRINTF("%s: maximum number of connections allowed exceeded (max %lu)\n",PHPFUNC, (unsigned long)LIBVIRT_G(max_connections_ini));
         set_error("Maximum number of connections allowed exceeded" TSRMLS_CC);
         RETURN_FALSE;
     }
@@ -9755,11 +9746,11 @@ PHP_FUNCTION(libvirt_print_binding_resources)
     for (i = 0; i < binding_resources_count; i++) {
         if (binding_resources[i].overwrite == 0) {
             if (binding_resources[i].conn != NULL)
-                snprintf(tmp, sizeof(tmp), "Libvirt %s resource at 0x%"UINTx" (connection %p)", translate_counter_type(binding_resources[i].type),
-                         binding_resources[i].mem, binding_resources[i].conn);
+                snprintf(tmp, sizeof(tmp), "Libvirt %s resource at 0x%lx (connection %lx)", translate_counter_type(binding_resources[i].type),
+                         (long)binding_resources[i].mem, (long)binding_resources[i].conn);
             else
-                snprintf(tmp, sizeof(tmp), "Libvirt %s resource at 0x%"UINTx, translate_counter_type(binding_resources[i].type),
-                         binding_resources[i].mem);
+                snprintf(tmp, sizeof(tmp), "Libvirt %s resource at 0x%lx", translate_counter_type(binding_resources[i].type),
+                         (long)binding_resources[i].mem);
             add_next_index_string(return_value, tmp VIRT_COPY_OPT);
         }
     }
diff --git a/src/libvirt-php.h b/src/libvirt-php.h
index d1f034e..09b3868 100644
--- a/src/libvirt-php.h
+++ b/src/libvirt-php.h
@@ -166,7 +166,7 @@ typedef struct tTokenizer {
 typedef struct _resource_info {
     int type;
     virConnectPtr conn;
-    arch_uint mem;
+    void *mem;
     int overwrite;
 } resource_info;
 
@@ -176,7 +176,7 @@ ZEND_BEGIN_MODULE_GLOBALS(libvirt)
     zend_bool longlong_to_string_ini;
     char *iso_path_ini;
     char *image_path_ini;
-    char *max_connections_ini;
+    zend_long max_connections_ini;
 #ifdef DEBUG_SUPPORT
     int debug;
 #endif
-- 
2.5.5

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]