Changeset 328 for branches/1.2/xcache.c

Timestamp:

2006-12-16T12:56:30+01:00 (6 years ago)

Author:

moo

Message:

1.2: fixed #56: xcache_set segfaults when xcache.var_size=0

File:

• branches/1.2/xcache.c (modified) (6 diffs)

branches/1.2/xcache.c

@@ -1696 +1696 @@
 /* }}} */
 
+#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"); \
+} while (0)
+
 static int xc_entry_init_key_var(xc_entry_t *xce, zval *name TSRMLS_DC) /* {{{ */
 {
@@ -1737 +1741 @@
     xc_entry_data_var_t var;
     zval *name;
+
+    if (!xc_var_caches) {
+        VAR_DISABLED_WARNING();
+        RETURN_NULL();
+    }
 
     if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &name) == FAILURE) {
@@ -1770 +1779 @@
     zval *value;
 
+    if (!xc_var_caches) {
+        VAR_DISABLED_WARNING();
+        RETURN_NULL();
+    }
+
     xce.ttl = XG(var_ttl);
     if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "zz|l", &name, &value, &xce.ttl) == FAILURE) {
@@ -1800 +1814 @@
     xc_entry_data_var_t var;
     zval *name;
+
+    if (!xc_var_caches) {
+        VAR_DISABLED_WARNING();
+        RETURN_FALSE;
+    }
 
     if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &name) == FAILURE) {
@@ -1832 +1851 @@
     zval *name;
 
+    if (!xc_var_caches) {
+        VAR_DISABLED_WARNING();
+        RETURN_FALSE;
+    }
+
     if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &name) == FAILURE) {
         return;
@@ -1858 +1882 @@
     long value = 0;
     zval oldzval;
+
+    if (!xc_var_caches) {
+        VAR_DISABLED_WARNING();
+        RETURN_NULL();
+    }
 
     xce.ttl = XG(var_ttl);