Revert

Never mind, it looks like ext/filter does an estrdup on everything it
gets which doesn't seem very efficient to me as most things it gets
passed will already be emalloc'ed.  My custom version of the filter
extension doesn't do this which led to the confusion.

diff --git a/sapi/apache/mod_php5.c b/sapi/apache/mod_php5.c
index 0845f8e40b54ec3be5a9df0bee92ea3b77e7eaea..d84b11e548e29b8d1b39ab1a45526aef8101023a 100644 (file)
--- a/sapi/apache/mod_php5.c
+++ b/sapi/apache/mod_php5.c
@@ -243,12 +243,14 @@ static void sapi_apache_register_server_variables(zval *track_vars_array TSRMLS_
        table_entry *elts = (table_entry *) arr->elts;
        zval **path_translated;
        HashTable *symbol_table;
-       int val_len, new_val_len;
-       char *val;
+       int new_val_len;
 
        for (i = 0; i < arr->nelts; i++) {
+               char *val;
+               int val_len;
+
                if (elts[i].val) {
-                       val = estrdup(elts[i].val);
+                       val = elts[i].val;
                } else {
                        val = "";
                }
@@ -273,9 +275,8 @@ static void sapi_apache_register_server_variables(zval *track_vars_array TSRMLS_
                php_register_variable("PATH_TRANSLATED", Z_STRVAL_PP(path_translated), track_vars_array TSRMLS_CC);
        }
 
-       val = estrdup(((request_rec *)SG(server_context))->uri);
-       if (sapi_module.input_filter(PARSE_SERVER, "PHP_SELF", val, strlen(val), &new_val_len TSRMLS_CC)) {
-               php_register_variable_safe("PHP_SELF", val, new_val_len, track_vars_array TSRMLS_CC);
+       if (sapi_module.input_filter(PARSE_SERVER, "PHP_SELF", &((request_rec *) SG(server_context))->uri, strlen(((request_rec *) SG(server_context))->uri), &new_val_len TSRMLS_CC)) {
+               php_register_variable("PHP_SELF", ((request_rec *) SG(server_context))->uri, track_vars_array TSRMLS_CC);
        }
 }
 /* }}} */