Changeset 960 for trunk/xcache.c

Timestamp:

2012-07-09T10:38:25+02:00 (11 months ago)

Author:

moo

Message:

fixed #283: XCache admin APIs (admin page) cause SEGV if XCache failed to initize

File:

• trunk/xcache.c (modified) (3 diffs)

trunk/xcache.c

@@ -2731 +2731 @@
     switch (optype) {
         case XC_OP_COUNT:
-            RETURN_LONG(size)
+            RETURN_LONG(caches ? size : 0)
             break;
 
         case XC_OP_INFO:
         case XC_OP_LIST:
-            if (id < 0 || id >= size) {
+            if (!caches || id < 0 || id >= size) {
                 php_error_docref(NULL TSRMLS_CC, E_WARNING, "Cache not exists");
                 RETURN_FALSE;
@@ -2758 +2758 @@
                 int entryslotid, c;
 
-                if (id < 0 || id >= size) {
+                if (!caches || id < 0 || id >= size) {
                     php_error_docref(NULL TSRMLS_CC, E_WARNING, "Cache not exists");
                     RETURN_FALSE;
@@ -2812 +2812 @@
 
 #define VAR_DISABLED_WARNING() do { \
-        php_error_docref(NULL TSRMLS_CC, E_WARNING, "xcache.var_size is either 0 or too small to enable var data caching"); \
+        php_error_docref(NULL TSRMLS_CC, E_WARNING, "XCache var cache was not initialized properly. Check php log for actual reason"); \
 } while (0)