dmitry Wed Feb 2 04:11:03 2005 EDT
Added files: (Branch: PHP_5_0)
/php-src/ext/soap/tests/bugs bug31755.phpt
Modified files:
/php-src NEWS
/php-src/ext/soap php_http.c soap.c
Log:
Fixed bug #31755 (Cannot create SOAP header in no namespace)
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.226&r2=1.1760.2.227&ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.226 php-src/NEWS:1.1760.2.227
--- php-src/NEWS:1.1760.2.226 Wed Feb 2 02:19:01 2005
+++ php-src/NEWS Wed Feb 2 04:11:02 2005
@@ -16,6 +16,7 @@
- Fixed posix_getsid() & posix_getpgid() to return sid & pgid instead
of true. (Tony)
- Fixed bug #31797 (exif_read_data() uses too low nesting limit). (Ilia)
+- Fixed bug #31755 (Cannot create SOAP header in no namespace). (Dmitry)
- Fixed bug #31732 (mb_get_info() causes segfault when no parameters
specified). (Tony)
- Fixed bug #31710 (Wrong return values for mysqli_autocommit/commit/rollback).
http://cvs.php.net/diff.php/php-src/ext/soap/php_http.c?r1=1.55.2.10&r2=1.55.2.11&ty=u
Index: php-src/ext/soap/php_http.c
diff -u php-src/ext/soap/php_http.c:1.55.2.10
php-src/ext/soap/php_http.c:1.55.2.11
--- php-src/ext/soap/php_http.c:1.55.2.10 Tue Jan 25 06:02:45 2005
+++ php-src/ext/soap/php_http.c Wed Feb 2 04:11:03 2005
@@ -17,7 +17,7 @@
| Dmitry Stogov <[EMAIL PROTECTED]> |
+----------------------------------------------------------------------+
*/
-/* $Id: php_http.c,v 1.55.2.10 2005/01/25 11:02:45 dmitry Exp $ */
+/* $Id: php_http.c,v 1.55.2.11 2005/02/02 09:11:03 dmitry Exp $ */
#include "php_soap.h"
#include "ext/standard/base64.h"
@@ -655,6 +655,7 @@
}
if (!get_http_headers(stream, &http_headers, &http_header_size
TSRMLS_CC)) {
+ efree(http_headers);
if (request != buf) {efree(request);}
php_stream_close(stream);
zend_hash_del(Z_OBJPROP_P(this_ptr), "httpsocket",
sizeof("httpsocket"));
@@ -690,6 +691,7 @@
if (http_status == 100) {
efree(http_headers);
if (!get_http_headers(stream, &http_headers,
&http_header_size TSRMLS_CC)) {
+ efree(http_headers);
if (request != buf) {efree(request);}
php_stream_close(stream);
zend_hash_del(Z_OBJPROP_P(this_ptr),
"httpsocket", sizeof("httpsocket"));
http://cvs.php.net/diff.php/php-src/ext/soap/soap.c?r1=1.110.2.22&r2=1.110.2.23&ty=u
Index: php-src/ext/soap/soap.c
diff -u php-src/ext/soap/soap.c:1.110.2.22 php-src/ext/soap/soap.c:1.110.2.23
--- php-src/ext/soap/soap.c:1.110.2.22 Mon Jan 31 09:26:24 2005
+++ php-src/ext/soap/soap.c Wed Feb 2 04:11:03 2005
@@ -17,7 +17,7 @@
| Dmitry Stogov <[EMAIL PROTECTED]> |
+----------------------------------------------------------------------+
*/
-/* $Id: soap.c,v 1.110.2.22 2005/01/31 14:26:24 dmitry Exp $ */
+/* $Id: soap.c,v 1.110.2.23 2005/02/02 09:11:03 dmitry Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -656,6 +656,9 @@
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "zs", &data,
&name, &name_length) == FAILURE) {
php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters");
}
+ if (name_length == 0) {
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters.
Invalid parameter name.");
+ }
#ifndef ZEND_ENGINE_2
zval_add_ref(&data);
@@ -678,6 +681,12 @@
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|zbz", &ns,
&ns_len, &name, &name_len, &data, &must_understand, &actor) == FAILURE) {
php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters");
}
+ if (ns_len == 0) {
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters.
Invalid namespace.");
+ }
+ if (name_len == 0) {
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters.
Invalid header name.");
+ }
add_property_stringl(this_ptr, "namespace", ns, ns_len, 1);
add_property_stringl(this_ptr, "name", name, name_len, 1);
@@ -689,12 +698,15 @@
}
add_property_bool(this_ptr, "mustUnderstand", must_understand);
if (actor == NULL) {
- } else if (Z_TYPE_P(actor) == IS_LONG) {
+ } else if (Z_TYPE_P(actor) == IS_LONG &&
+ (Z_LVAL_P(actor) == SOAP_ACTOR_NEXT ||
+ Z_LVAL_P(actor) == SOAP_ACTOR_NONE ||
+ Z_LVAL_P(actor) == SOAP_ACTOR_UNLIMATERECEIVER)) {
add_property_long(this_ptr, "actor", Z_LVAL_P(actor));
- } else if (Z_TYPE_P(actor) == IS_STRING) {
+ } else if (Z_TYPE_P(actor) == IS_STRING && Z_STRLEN_P(actor) > 0) {
add_property_stringl(this_ptr, "actor", Z_STRVAL_P(actor),
Z_STRLEN_P(actor), 1);
} else {
- php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters");
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters.
Invalid actor.");
}
}
@@ -713,6 +725,12 @@
&details, &name, &name_len, &headerfault) == FAILURE) {
php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters");
}
+ if (fault_code != NULL && fault_code_len == 0) {
+ php_error_docref(NULL TSRMLS_CC, E_ERROR, "Invalid parameters.
Invalid fault code.");
+ }
+ if (name != NULL && name_len == 0) {
+ name = NULL;
+ }
set_soap_fault(this_ptr, fault_code, fault_string, fault_actor,
details, name TSRMLS_CC);
if (headerfault != NULL) {
@@ -792,16 +810,16 @@
add_property_zval(this_ptr, "enc_value", data);
}
- if (stype && strlen(stype) > 0) {
+ if (stype && stype_len > 0) {
add_property_stringl(this_ptr, "enc_stype", stype, stype_len,
1);
}
- if (ns && strlen(ns) > 0) {
+ if (ns && ns_len > 0) {
add_property_stringl(this_ptr, "enc_ns", ns, ns_len, 1);
}
- if (name && strlen(name) > 0) {
+ if (name && name_len > 0) {
add_property_stringl(this_ptr, "enc_name", name, name_len, 1);
}
- if (namens && strlen(namens) > 0) {
+ if (namens && namens_len > 0) {
add_property_stringl(this_ptr, "enc_namens", namens,
namens_len, 1);
}
}
http://cvs.php.net/co.php/php-src/ext/soap/tests/bugs/bug31755.phpt?r=1.1&p=1
Index: php-src/ext/soap/tests/bugs/bug31755.phpt
+++ php-src/ext/soap/tests/bugs/bug31755.phpt
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
