diff options
author | Philip Hofstetter <phofstetter@sensational.ch> | 2014-09-03 14:35:40 +0200 |
---|---|---|
committer | Philip Hofstetter <phofstetter@sensational.ch> | 2014-09-03 14:35:40 +0200 |
commit | 9befa8c5f7570808dbf31004ea33534cd61b7383 (patch) | |
tree | 615e1c3c122710cbc9b1d21ebc666176c915ebae | |
parent | 7ccbfb2e8467f4a5bec90209254f78ffb948755e (diff) | |
download | php-git-9befa8c5f7570808dbf31004ea33534cd61b7383.tar.gz |
fix bug #67955
this fixes a regression from 6c2a8068207a02b3d7ae7416a9967dad0a81e61f.
smart_str_appendl is expecting the length as the length of the string,
but key_length is the byte length of the key, including the 0
terminator.
As such, the cookie name appeneded to the header would now also include
the 0 terminator of the key name which then would be sent to the server.
-rw-r--r-- | ext/soap/php_http.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ext/soap/php_http.c b/ext/soap/php_http.c index 2a5679a439..4a5829fadb 100644 --- a/ext/soap/php_http.c +++ b/ext/soap/php_http.c @@ -830,7 +830,7 @@ try_again: (zend_hash_index_find(Z_ARRVAL_PP(data), 2, (void**)&tmp) == FAILURE || in_domain(phpurl->host,Z_STRVAL_PP(tmp))) && (use_ssl || zend_hash_index_find(Z_ARRVAL_PP(data), 3, (void**)&tmp) == FAILURE)) { - smart_str_appendl(&soap_headers, key, key_len); + smart_str_appendl(&soap_headers, key, key_len-1); smart_str_appendc(&soap_headers, '='); smart_str_appendl(&soap_headers, Z_STRVAL_PP(value), Z_STRLEN_PP(value)); smart_str_appendc(&soap_headers, ';'); |