diff options
author | Michael Wallner <mike@php.net> | 2014-09-09 06:56:33 +0200 |
---|---|---|
committer | Michael Wallner <mike@php.net> | 2014-09-09 06:56:33 +0200 |
commit | a6c47caf25255669f68c30104fc9c5fbc9d1cbc8 (patch) | |
tree | fd7d8277c133d8a6799d24b0f3ecb4b717466378 /main/php_variables.c | |
parent | 2470e01ab6d30e5ed3d890f0aaf1321215edb4cd (diff) | |
parent | 8d1099ac0574f3a42036085641c2df03a1d5f731 (diff) | |
download | php-git-a6c47caf25255669f68c30104fc9c5fbc9d1cbc8.tar.gz |
Merge branch 'PHP-5.6'
* PHP-5.6:
duplicate value's string for the SAPI filter
Fine tuned the order of adding request variables.
Diffstat (limited to 'main/php_variables.c')
-rw-r--r-- | main/php_variables.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/main/php_variables.c b/main/php_variables.c index 37af78b98d..0f16b92d02 100644 --- a/main/php_variables.c +++ b/main/php_variables.c @@ -243,7 +243,7 @@ typedef struct post_var_data { static zend_bool add_post_var(zval *arr, post_var_data_t *var, zend_bool eof TSRMLS_DC) { - char *ksep, *vsep; + char *ksep, *vsep, *val; size_t klen, vlen; /* FIXME: string-size_t */ size_t new_vlen; @@ -274,15 +274,17 @@ static zend_bool add_post_var(zval *arr, post_var_data_t *var, zend_bool eof TSR vlen = 0; } - php_url_decode(var->ptr, klen); + + val = estrndup(ksep, vlen); if (vlen) { - vlen = php_url_decode(ksep, vlen); + vlen = php_url_decode(val, vlen); } - if (sapi_module.input_filter(PARSE_POST, var->ptr, &ksep, vlen, &new_vlen TSRMLS_CC)) { - php_register_variable_safe(var->ptr, ksep, new_vlen, arr TSRMLS_CC); + if (sapi_module.input_filter(PARSE_POST, var->ptr, &val, vlen, &new_vlen TSRMLS_CC)) { + php_register_variable_safe(var->ptr, val, new_vlen, arr TSRMLS_CC); } + efree(val); var->ptr = vsep + (vsep != var->end); return 1; |