MFH: Fixed bug #46746 (xmlrpc_decode_request outputs non-suppressable error

when given bad data).

diff --git a/NEWS b/NEWS
index 520094e9ce16bb3b5c3e92a968935f24c934a136..5d272cb6ff3bcc97ee644e2aa3a3bf7ca52de363 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,8 @@ PHP                                                                        NEWS
   session_is_registered(). (Hannes)
 
 - Fixed bug #46811 (ini_set() doesn't return false on failure). (Hannes)
+- Fixed bug #46746 (xmlrpc_decode_request outputs non-suppressable error when
+  given bad data). (Ilia)
 
 04 Dec 2008, PHP 5.3.0 Alpha 3
 - Upgraded bundled PCRE to version 7.8. (Nuno)

diff --git a/ext/xmlrpc/libxmlrpc/xml_element.c b/ext/xmlrpc/libxmlrpc/xml_element.c
index 0ae08db7d5296c7ed09a9aadc4ae8f18a2f0a2af..3762f80080b99dfbf09420052c52bac006e56621 100644 (file)
--- a/ext/xmlrpc/libxmlrpc/xml_element.c
+++ b/ext/xmlrpc/libxmlrpc/xml_element.c
@@ -44,6 +44,9 @@ static const char rcsid[] = "#(@) $Id$";
  *   06/2000
  * HISTORY
  *   $Log$
+ *   Revision 1.9.4.1  2006/07/30 11:34:02  tony2001
+ *   MFH: fix compile warnings (#38257)
+ *
  *   Revision 1.9  2005/04/22 11:06:53  jorton
  *   Fixed bug #32797 (invalid C code in xmlrpc extension).
  *
@@ -375,7 +378,7 @@ static void xml_element_serialize(xml_element *el, int (*fptr)(void *data, const
    depth++;
 
    if(!el) {
-      fprintf(stderr, "Nothing to write\n");
+      /* fprintf(stderr, "Nothing to write\n"); */
       return;
    }
    if(!options) {
@@ -722,7 +725,7 @@ xml_element* xml_elem_parse_buf(const char* in_buf, int len, XML_ELEM_INPUT_OPTI
                       byte_idx > 10  ? 10 : byte_idx,
                       in_buf + (byte_idx > 10 ? byte_idx - 10 : byte_idx));
          }
-
+/*
          fprintf(stderr, "expat reports error code %i\n"
                 "\tdescription: %s\n"
                 "\tline: %i\n"
@@ -731,7 +734,7 @@ xml_element* xml_elem_parse_buf(const char* in_buf, int len, XML_ELEM_INPUT_OPTI
                 "\ttotal bytes: %i\n%s ",
                 err_code, error_str, line_num, 
                 col_num, byte_idx, byte_total, buf);
-
+*/
 
           /* error condition */
           if(error) {

diff --git a/ext/xmlrpc/libxmlrpc/xmlrpc.c b/ext/xmlrpc/libxmlrpc/xmlrpc.c
index 7b27caee3e466c3a7090fda6badfac2c1c3874d6..ec2321b347e333596e22065a9d0f22b2c3906e80 100644 (file)
--- a/ext/xmlrpc/libxmlrpc/xmlrpc.c
+++ b/ext/xmlrpc/libxmlrpc/xmlrpc.c
@@ -43,6 +43,13 @@ static const char rcsid[] = "#(@) $Id$";
  *   9/1999 - 10/2000
  * HISTORY
  *   $Log$
+ *   Revision 1.8.4.3.2.1  2008/09/10 00:07:44  felipe
+ *   MFH:
+ *   - Merged fix from SF project (Import Jeff Lawsons patches for XML datetime bug fixes)
+ *     Fixed bugs:
+ *     #45226 (xmlrpc_set_type() segfaults with valid ISO8601 date string)
+ *     #18916 (xmlrpc_set_type() "not working")
+ *
  *   Revision 1.8.4.3  2007/09/18 19:49:53  iliaa
  *
  *   Fixed bug #42189 (xmlrpc_set_type() crashes php on invalid datetime
@@ -1161,13 +1168,13 @@ int XMLRPC_AddValueToVector(XMLRPC_VALUE target, XMLRPC_VALUE source) {
                   }
                }
                else {
-                                       fprintf (stderr,
-                                                               "xmlrpc: attempted to add key/val pair to vector of type array\n");
+                                       /* fprintf (stderr,
+                                                               "xmlrpc: attempted to add key/val pair to vector of type array\n"); */
                }
                break;
             default:
-                               fprintf (stderr,
-                                                       "xmlrpc: attempted to add value of unknown type to vector\n");
+                               /* fprintf (stderr,
+                                                       "xmlrpc: attempted to add value of unknown type to vector\n"); */
                break;
          }
       }
@@ -1497,8 +1504,8 @@ void XMLRPC_CleanupValue(XMLRPC_VALUE value) {
                my_free(value);
                break;
             default:
-                               fprintf (stderr,
-                                                       "xmlrpc: attempted to free value of invalid type\n");
+                               /* fprintf (stderr,
+                                                       "xmlrpc: attempted to free value of invalid type\n"); */
                break;
          }
       }