[PHP-CVS-DAILY] cvs: php-src / ChangeLog

2005-11-18 Thread changelog
changelog   Sat Nov 19 01:53:07 2005 EDT

  Modified files:  
/php-srcChangeLog 
  Log:
  ChangeLog update
  
http://cvs.php.net/diff.php/php-src/ChangeLog?r1=1.2168r2=1.2169ty=u
Index: php-src/ChangeLog
diff -u php-src/ChangeLog:1.2168 php-src/ChangeLog:1.2169
--- php-src/ChangeLog:1.2168Fri Nov 18 01:42:17 2005
+++ php-src/ChangeLog   Sat Nov 19 01:52:43 2005
@@ -1,3 +1,141 @@
+2005-11-18  Frank M. Kromann  [EMAIL PROTECTED]
+
+* (PHP_4_4)
+  NEWS
+  NEWS
+  NEWS
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c:
+  MFH: Fix #33201 Crash when fetching some data types
+
+* NEWS
+  ext/mssql/php_mssql.c:
+  Fix #33201 Crash when fetching some data types
+
+* (PHP_4_4)
+  NEWS
+  NEWS
+  NEWS
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c:
+  MFH Fix #33963. mssql_bind fails on input parameters
+
+* NEWS
+  ext/mssql/php_mssql.c:
+  Fix #33963. mssql_bind fails on input parameters
+
+* (PHP_4_4)
+  NEWS
+  NEWS
+  NEWS
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c:
+  MFH: Fix #32009 crash when mssql_bind() is called more than once
+
+* NEWS
+  ext/mssql/php_mssql.c:
+  Fix #32009 crash when mssql_bind() is called more than once
+
+* (PHP_4_4)
+  NEWS
+  NEWS
+  NEWS
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c
+  ext/mssql/php_mssql.c:
+  MFH: Fix #33153 Crash in mssql_next_result().
+
+* NEWS
+  ext/mssql/php_mssql.c:
+  Fix #33153 Crash in mssql_next_result().
+
+2005-11-18  Ilia Alshanetsky  [EMAIL PROTECTED]
+
+* (PHP_4_4)
+  NEWS
+  sapi/apache2handler/sapi_apache2.c:
+  MFH: Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php 
module).
+
+* sapi/apache2handler/sapi_apache2.c:
+  MFB51: Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php
+  module)
+
+* (PHP_5_1)
+  NEWS
+  sapi/apache2handler/sapi_apache2.c:
+  Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module).
+
+2005-11-18  Jani Taskinen  [EMAIL PROTECTED]
+
+* NEWS:
+  reorder
+
+2005-11-18  Rasmus Lerdorf  [EMAIL PROTECTED]
+
+* NEWS:
+  allow_url_include
+
+* php.ini-dist
+  php.ini-recommended
+  main/main.c
+  main/php_globals.h
+  main/streams/streams.c:
+  Add allow_url_include to let people turn on allow_url_fopen without also
+  enabling remote code execution through url wrappers
+
+2005-11-18  Jani Taskinen  [EMAIL PROTECTED]
+
+* (PHP_5_1)
+  INSTALL:
+  php4 - php5
+
+* ext/readline/config.m4:
+  MFB51: fix build when --with-libedit is used
+
+* (PHP_4_4)
+  ext/readline/config.m4:
+  MF51: sync all changes
+
+* (PHP_5_1)
+  ext/readline/config.m4:
+  - Make sure --with-readline is used only when --with-libedit is not used.
+
+* (PHP_4_4)
+  NEWS:
+  fix news
+
+2005-11-18  Derick Rethans  [EMAIL PROTECTED]
+
+* (PHP_4_4)
+  NEWS
+  configure.in
+  main/php_version.h:
+  - Back to -dev.
+
+2005-11-18  Dmitry Stogov  [EMAIL PROTECTED]
+
+* (PHP_5_0)
+  NEWS
+  NEWS:
+  Fixed bug #35273 (Error in mapping soap - java types)
+
+2005-11-18  Derick Rethans  [EMAIL PROTECTED]
+
+* (PHP_4_4)
+  NEWS
+  configure.in
+  main/php_version.h:
+  - Go with 4.4.2RC1.
+
+2005-11-18  Jani Taskinen  [EMAIL PROTECTED]
+
+* (PHP_5_1)
+  configure.in
+  main/php_version.h:
+  back to dev
+
 2005-11-17  Marcus Boerger  [EMAIL PROTECTED]
 
 * ext/standard/credits_ext.h:
@@ -16691,7 +16829,7 @@
 
 * (PHP_4_3)
   ext/standard/url_scanner_ex.re:
-  Missing $Id: ChangeLog,v 1.2168 2005/11/18 06:42:17 changelog Exp $ tag
+  Missing $Id: ChangeLog,v 1.2169 2005/11/19 06:52:43 changelog Exp $ tag
 
 * (PHP_5_0)
   ext/standard/url_scanner_ex.c:
@@ -20092,7 +20230,7 @@
   ext/session/mod_mm.h
   ext/session/mod_user.h
   ext/session/php_session.h:
-  - Missing $Id: ChangeLog,v 1.2168 2005/11/18 06:42:17 changelog Exp $ 
tags
+  - Missing $Id: ChangeLog,v 1.2169 2005/11/19 06:52:43 changelog Exp $ 
tags
 
 * (PHP_4_3)
   ext/session/session.c:
@@ -20573,7 +20711,7 @@
 
 * (PHP_4_3)
   Zend/zend_ini_scanner.l:
-  Missing $Id: ChangeLog,v 1.2168 2005/11/18 06:42:17 changelog Exp $ tag 
+ ws fix
+  Missing $Id: ChangeLog,v 1.2169 2005/11/19 06:52:43 changelog Exp $ tag 
+ ws fix
 
 * ZendEngine2/zend_ini_scanner.l
   ZendEngine2/zend_ini_scanner.l:
@@ -22047,7 +22185,7 @@
   ext/standard/url_scanner_ex.c
   ext/standard/url_scanner_ex.h
   ext/standard/url_scanner_ex.re:
-  Missing $Id: ChangeLog,v 1.2168 2005/11/18 06:42:17 changelog Exp $ tag
+  Missing $Id: ChangeLog,v 1.2169 2005/11/19 06:52:43 changelog Exp $ tag
 
 * ext/standard/credits_ext.h
   

Re: [PHP-CVS] cvs: php-src(PHP_5_1) /ext/reflection php_reflection_api.c php_reflection_api.h

2005-11-18 Thread Jani Taskinen


Why not? IMO, it's the same if it's in 5.1 or HEAD, nothing changes
but the location of the sources, right? It will be hell for you
to merge fixes from HEAD now..

--Jani

On Thu, 17 Nov 2005, Marcus Boerger wrote:



helly   Thu Nov 17 17:35:27 2005 EDT

 Removed files:   (Branch: PHP_5_1)
   /php-src/ext/reflection  php_reflection_api.c php_reflection_api.h
 Log:
 - Not in this branch





--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Derick Rethans
On Thu, 17 Nov 2005, Marcus Boerger wrote:

 helly Thu Nov 17 18:21:15 2005 EDT
 
   Modified files:  
 /php-src/ext/reflection   config.m4 
   Log:
   - Need to define HAVE_REFLECTION

I think it should not be possible to disable this extension with a 
simple configure switch...

Derick

-- 
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / configure.in /main php_version.h

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 04:51:44 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcconfigure.in 
/php-src/main   php_version.h 
  Log:
  back to dev
  
http://cvs.php.net/diff.php/php-src/configure.in?r1=1.579.2.18r2=1.579.2.19ty=u
Index: php-src/configure.in
diff -u php-src/configure.in:1.579.2.18 php-src/configure.in:1.579.2.19
--- php-src/configure.in:1.579.2.18 Thu Nov 17 16:27:56 2005
+++ php-src/configure.inFri Nov 18 04:51:39 2005
@@ -1,4 +1,4 @@
- ## $Id: configure.in,v 1.579.2.18 2005/11/17 21:27:56 iliaa Exp $ -*- 
autoconf -*-
+ ## $Id: configure.in,v 1.579.2.19 2005/11/18 09:51:39 sniper Exp $ -*- 
autoconf -*-
 dnl ## Process this file with autoconf to produce a configure script.
 
 divert(1)
@@ -42,7 +42,7 @@
 MAJOR_VERSION=5
 MINOR_VERSION=1
 RELEASE_VERSION=0
-EXTRA_VERSION=RC6
+EXTRA_VERSION=RC7-dev
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$RELEASE_VERSION$EXTRA_VERSION
 
 dnl Define where extension directories are located in the configure context
http://cvs.php.net/diff.php/php-src/main/php_version.h?r1=1.112.2.14r2=1.112.2.15ty=u
Index: php-src/main/php_version.h
diff -u php-src/main/php_version.h:1.112.2.14 
php-src/main/php_version.h:1.112.2.15
--- php-src/main/php_version.h:1.112.2.14   Thu Nov 17 16:27:58 2005
+++ php-src/main/php_version.h  Fri Nov 18 04:51:44 2005
@@ -3,5 +3,5 @@
 #define PHP_MAJOR_VERSION 5
 #define PHP_MINOR_VERSION 1
 #define PHP_RELEASE_VERSION 0
-#define PHP_EXTRA_VERSION RC6
-#define PHP_VERSION 5.1.0RC6
+#define PHP_EXTRA_VERSION RC7-dev
+#define PHP_VERSION 5.1.0RC7-dev

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS configure.in /main php_version.h

2005-11-18 Thread Derick Rethans
derick  Fri Nov 18 05:30:46 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS configure.in 
/php-src/main   php_version.h 
  Log:
  - Go with 4.4.2RC1.
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.80r2=1.1247.2.920.2.81ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.80 php-src/NEWS:1.1247.2.920.2.81
--- php-src/NEWS:1.1247.2.920.2.80  Thu Nov 17 16:01:13 2005
+++ php-src/NEWSFri Nov 18 05:30:37 2005
@@ -1,6 +1,6 @@
 PHP 4  NEWS
 |||
-?? ??? 2006, Version 4.4.2
+18 Nov 2005, Version 4.4.2RC1
 - Resolve Apache 2 regression with sub-request handling on non-linux systems.
   (Ilia, Tony)
 - Backport missing imap_mailcompose fixes from 5.X. (Ilia)
http://cvs.php.net/diff.php/php-src/configure.in?r1=1.396.2.164.2.14r2=1.396.2.164.2.15ty=u
Index: php-src/configure.in
diff -u php-src/configure.in:1.396.2.164.2.14 
php-src/configure.in:1.396.2.164.2.15
--- php-src/configure.in:1.396.2.164.2.14   Sun Oct 30 05:55:21 2005
+++ php-src/configure.inFri Nov 18 05:30:40 2005
@@ -1,4 +1,4 @@
-dnl ## $Id: configure.in,v 1.396.2.164.2.14 2005/10/30 10:55:21 derick Exp $ 
-*- sh -*-
+dnl ## $Id: configure.in,v 1.396.2.164.2.15 2005/11/18 10:30:40 derick Exp $ 
-*- sh -*-
 dnl ## Process this file with autoconf to produce a configure script.
 
 divert(1)
@@ -41,7 +41,7 @@
 MAJOR_VERSION=4
 MINOR_VERSION=4
 RELEASE_VERSION=2
-EXTRA_VERSION=-dev
+EXTRA_VERSION=RC1
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$RELEASE_VERSION$EXTRA_VERSION
 
 dnl Define where extension directories are located in the configure context
http://cvs.php.net/diff.php/php-src/main/php_version.h?r1=1.66.2.81.2.11r2=1.66.2.81.2.12ty=u
Index: php-src/main/php_version.h
diff -u php-src/main/php_version.h:1.66.2.81.2.11 
php-src/main/php_version.h:1.66.2.81.2.12
--- php-src/main/php_version.h:1.66.2.81.2.11   Sun Oct 30 05:55:23 2005
+++ php-src/main/php_version.h  Fri Nov 18 05:30:45 2005
@@ -3,5 +3,5 @@
 #define PHP_MAJOR_VERSION 4
 #define PHP_MINOR_VERSION 4
 #define PHP_RELEASE_VERSION 2
-#define PHP_EXTRA_VERSION -dev
-#define PHP_VERSION 4.4.2-dev
+#define PHP_EXTRA_VERSION RC1
+#define PHP_VERSION 4.4.2RC1

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src /ext/soap php_sdl.c /ext/soap/tests/bugs bug35273.phpt

2005-11-18 Thread Dmitry Stogov
dmitry  Fri Nov 18 05:59:55 2005 EDT

  Added files: 
/php-src/ext/soap/tests/bugsbug35273.phpt 

  Modified files:  
/php-src/ext/soap   php_sdl.c 
  Log:
  Fixed bug #35273 (Error in mapping soap - java types)
  
  
http://cvs.php.net/diff.php/php-src/ext/soap/php_sdl.c?r1=1.89r2=1.90ty=u
Index: php-src/ext/soap/php_sdl.c
diff -u php-src/ext/soap/php_sdl.c:1.89 php-src/ext/soap/php_sdl.c:1.90
--- php-src/ext/soap/php_sdl.c:1.89 Mon Oct 24 03:43:48 2005
+++ php-src/ext/soap/php_sdl.c  Fri Nov 18 05:59:51 2005
@@ -17,7 +17,7 @@
   |  Dmitry Stogov [EMAIL PROTECTED] |
   +--+
 */
-/* $Id: php_sdl.c,v 1.89 2005/10/24 07:43:48 dmitry Exp $ */
+/* $Id: php_sdl.c,v 1.90 2005/11/18 10:59:51 dmitry Exp $ */
 
 #include php_soap.h
 #include ext/libxml/php_libxml.h
@@ -117,24 +117,40 @@
nscat[len] = '\0';
 
enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
 
if (enc == NULL 
((ns_len == sizeof(SOAP_1_1_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_1_ENC_NAMESPACE, 
sizeof(SOAP_1_1_ENC_NAMESPACE)-1) == 0) ||
 (ns_len == sizeof(SOAP_1_2_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_2_ENC_NAMESPACE, 
sizeof(SOAP_1_2_ENC_NAMESPACE)-1) == 0))) {
-   ns_len = sizeof(XSD_NAMESPACE)-1;
-   len = ns_len + type_len + 1;
-   nscat = emalloc(len + 1);
-   memcpy(nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
-   nscat[ns_len] = ':';
-   memcpy(nscat+ns_len+1, type, type_len);
-   nscat[len] = '\0';
-
-   enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
+   char *enc_nscat;
+   int enc_ns_len;
+   int enc_len;
+
+   enc_ns_len = sizeof(XSD_NAMESPACE)-1;
+   enc_len = enc_ns_len + type_len + 1;
+   enc_nscat = emalloc(enc_len + 1);
+   memcpy(enc_nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
+   enc_nscat[enc_ns_len] = ':';
+   memcpy(enc_nscat+enc_ns_len+1, type, type_len);
+   enc_nscat[enc_len] = '\0';
+
+   enc = get_encoder_ex(NULL, enc_nscat, enc_len);
+   efree(enc_nscat);
+   if (enc  sdl) {
+   encodePtr new_enc   = emalloc(sizeof(encode));
+   memcpy(new_enc, enc, sizeof(encode));
+   new_enc-details.ns = estrndup(ns, ns_len);
+   new_enc-details.type_str = 
estrdup(new_enc-details.type_str);
+   if (sdl-encoders == NULL) {
+   sdl-encoders = emalloc(sizeof(HashTable));
+   zend_hash_init(sdl-encoders, 0, NULL, 
delete_encoder, 0);
+   }
+   zend_hash_update(sdl-encoders, nscat, len + 1, 
new_enc, sizeof(encodePtr), NULL);
+   enc = new_enc;
+   }
}
+   efree(nscat);
return enc;
 }
 

http://cvs.php.net/co.php/php-src/ext/soap/tests/bugs/bug35273.phpt?r=1.1p=1
Index: php-src/ext/soap/tests/bugs/bug35273.phpt
+++ php-src/ext/soap/tests/bugs/bug35273.phpt
--TEST--
Bug #35273 Error in mapping soap - java types 
--SKIPIF--
?php require_once('skipif.inc'); ?
--FILE--
?php
class TestSoapClient extends SoapClient {
  function __doRequest($request, $location, $action, $version) {
echo $request;
exit;
}
}

ini_set(soap.wsdl_cache_enabled, 0);
$client = new TestSoapClient(dirname(__FILE__).'/bug32941.wsdl', array(trace 
= 1, 'exceptions' = 0));
$ahoj = $client-echoPerson(array(name=Name,surname=Surname));
echo ok\n;
?
--EXPECT--
?xml version=1.0 encoding=UTF-8?
SOAP-ENV:Envelope xmlns:SOAP-ENV=http://schemas.xmlsoap.org/soap/envelope/; 
xmlns:ns1=http://service; 
xmlns:SOAP-ENC=http://schemas.xmlsoap.org/soap/encoding/; 
xmlns:ns2=urn:service.EchoService 
xmlns:xsd=http://www.w3.org/2001/XMLSchema; 
xmlns:xsi=http://www.w3.org/2001/XMLSchema-instance; 
SOAP-ENV:encodingStyle=http://schemas.xmlsoap.org/soap/encoding/;SOAP-ENV:Bodyns1:echoPersonp
 xsi:type=ns2:Personname xsi:type=SOAP-ENC:stringName/namesurname 
xsi:type=SOAP-ENC:stringSurname/surname/p/ns1:echoPerson/SOAP-ENV:Body/SOAP-ENV:Envelope

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /ext/soap php_sdl.c /ext/soap/tests/bugs bug35273.phpt

2005-11-18 Thread Dmitry Stogov
dmitry  Fri Nov 18 06:00:17 2005 EDT

  Added files: (Branch: PHP_5_1)
/php-src/ext/soap/tests/bugsbug35273.phpt 

  Modified files:  
/php-srcNEWS 
/php-src/ext/soap   php_sdl.c 
  Log:
  Fixed bug #35273 (Error in mapping soap - java types)
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.201r2=1.2027.2.202ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.201 php-src/NEWS:1.2027.2.202
--- php-src/NEWS:1.2027.2.201   Thu Nov 17 16:15:19 2005
+++ php-src/NEWSFri Nov 18 06:00:14 2005
@@ -1,5 +1,8 @@
 PHPNEWS
 |||
+?? Nov 2005, PHP 5.1
+- Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
+
 17 Nov 2005, PHP 5.1 Release Candidate 6
 - Changed function parameter parsing to handle integers in a non-strict fashion
   and to emit an E_NOTICE on badly formed integer values. (Ilia)
http://cvs.php.net/diff.php/php-src/ext/soap/php_sdl.c?r1=1.88.2.1r2=1.88.2.2ty=u
Index: php-src/ext/soap/php_sdl.c
diff -u php-src/ext/soap/php_sdl.c:1.88.2.1 php-src/ext/soap/php_sdl.c:1.88.2.2
--- php-src/ext/soap/php_sdl.c:1.88.2.1 Mon Oct 24 03:44:17 2005
+++ php-src/ext/soap/php_sdl.c  Fri Nov 18 06:00:15 2005
@@ -17,7 +17,7 @@
   |  Dmitry Stogov [EMAIL PROTECTED] |
   +--+
 */
-/* $Id: php_sdl.c,v 1.88.2.1 2005/10/24 07:44:17 dmitry Exp $ */
+/* $Id: php_sdl.c,v 1.88.2.2 2005/11/18 11:00:15 dmitry Exp $ */
 
 #include php_soap.h
 #include ext/libxml/php_libxml.h
@@ -117,24 +117,40 @@
nscat[len] = '\0';
 
enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
 
if (enc == NULL 
((ns_len == sizeof(SOAP_1_1_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_1_ENC_NAMESPACE, 
sizeof(SOAP_1_1_ENC_NAMESPACE)-1) == 0) ||
 (ns_len == sizeof(SOAP_1_2_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_2_ENC_NAMESPACE, 
sizeof(SOAP_1_2_ENC_NAMESPACE)-1) == 0))) {
-   ns_len = sizeof(XSD_NAMESPACE)-1;
-   len = ns_len + type_len + 1;
-   nscat = emalloc(len + 1);
-   memcpy(nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
-   nscat[ns_len] = ':';
-   memcpy(nscat+ns_len+1, type, type_len);
-   nscat[len] = '\0';
-
-   enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
+   char *enc_nscat;
+   int enc_ns_len;
+   int enc_len;
+
+   enc_ns_len = sizeof(XSD_NAMESPACE)-1;
+   enc_len = enc_ns_len + type_len + 1;
+   enc_nscat = emalloc(enc_len + 1);
+   memcpy(enc_nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
+   enc_nscat[enc_ns_len] = ':';
+   memcpy(enc_nscat+enc_ns_len+1, type, type_len);
+   enc_nscat[enc_len] = '\0';
+
+   enc = get_encoder_ex(NULL, enc_nscat, enc_len);
+   efree(enc_nscat);
+   if (enc  sdl) {
+   encodePtr new_enc   = emalloc(sizeof(encode));
+   memcpy(new_enc, enc, sizeof(encode));
+   new_enc-details.ns = estrndup(ns, ns_len);
+   new_enc-details.type_str = 
estrdup(new_enc-details.type_str);
+   if (sdl-encoders == NULL) {
+   sdl-encoders = emalloc(sizeof(HashTable));
+   zend_hash_init(sdl-encoders, 0, NULL, 
delete_encoder, 0);
+   }
+   zend_hash_update(sdl-encoders, nscat, len + 1, 
new_enc, sizeof(encodePtr), NULL);
+   enc = new_enc;
+   }
}
+   efree(nscat);
return enc;
 }
 

http://cvs.php.net/co.php/php-src/ext/soap/tests/bugs/bug35273.phpt?r=1.1p=1
Index: php-src/ext/soap/tests/bugs/bug35273.phpt
+++ php-src/ext/soap/tests/bugs/bug35273.phpt
--TEST--
Bug #35273 Error in mapping soap - java types 
--SKIPIF--
?php require_once('skipif.inc'); ?
--FILE--
?php
class TestSoapClient extends SoapClient {
  function __doRequest($request, $location, $action, $version) {
echo $request;
exit;
}
}

ini_set(soap.wsdl_cache_enabled, 0);
$client = new TestSoapClient(dirname(__FILE__).'/bug32941.wsdl', array(trace 
= 1, 'exceptions' = 0));
$ahoj = $client-echoPerson(array(name=Name,surname=Surname));
echo ok\n;
?
--EXPECT--
?xml version=1.0 encoding=UTF-8?
SOAP-ENV:Envelope xmlns:SOAP-ENV=http://schemas.xmlsoap.org/soap/envelope/; 
xmlns:ns1=http://service; 
xmlns:SOAP-ENC=http://schemas.xmlsoap.org/soap/encoding/; 
xmlns:ns2=urn:service.EchoService 
xmlns:xsd=http://www.w3.org/2001/XMLSchema; 
xmlns:xsi=http://www.w3.org/2001/XMLSchema-instance; 

[PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/soap php_sdl.c /ext/soap/tests/bugs bug35273.phpt

2005-11-18 Thread Dmitry Stogov
dmitry  Fri Nov 18 06:01:05 2005 EDT

  Added files: (Branch: PHP_5_0)
/php-src/ext/soap/tests/bugsbug35273.phpt 

  Modified files:  
/php-srcNEWS 
/php-src/ext/soap   php_sdl.c 
  Log:
  Fixed bug #35273 (Error in mapping soap - java types)
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.515r2=1.1760.2.516ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.515 php-src/NEWS:1.1760.2.516
--- php-src/NEWS:1.1760.2.515   Thu Nov 17 09:20:03 2005
+++ php-src/NEWSFri Nov 18 06:01:04 2005
@@ -4,6 +4,7 @@
 - Fixed an error in mysqli_fetch_fields (returned NULL instead of an
   array when row number  field_count). (Georg)
 - Renamed CachingRecursiveIterator to RecursiveCachingIterator. (Marcus)
+- Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
 - Fixed bug #35239 (Objects can lose references). (Dmitry)
 - Fixed bug #35229 (call_user_func() crashes when arguement_stack is nearly
   full). (Dmitry)
http://cvs.php.net/diff.php/php-src/ext/soap/php_sdl.c?r1=1.70.2.13r2=1.70.2.14ty=u
Index: php-src/ext/soap/php_sdl.c
diff -u php-src/ext/soap/php_sdl.c:1.70.2.13 
php-src/ext/soap/php_sdl.c:1.70.2.14
--- php-src/ext/soap/php_sdl.c:1.70.2.13Mon Oct 24 03:45:49 2005
+++ php-src/ext/soap/php_sdl.c  Fri Nov 18 06:01:05 2005
@@ -17,7 +17,7 @@
   |  Dmitry Stogov [EMAIL PROTECTED] |
   +--+
 */
-/* $Id: php_sdl.c,v 1.70.2.13 2005/10/24 07:45:49 dmitry Exp $ */
+/* $Id: php_sdl.c,v 1.70.2.14 2005/11/18 11:01:05 dmitry Exp $ */
 
 #include php_soap.h
 #include ext/libxml/php_libxml.h
@@ -117,24 +117,40 @@
nscat[len] = '\0';
 
enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
 
if (enc == NULL 
((ns_len == sizeof(SOAP_1_1_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_1_ENC_NAMESPACE, 
sizeof(SOAP_1_1_ENC_NAMESPACE)-1) == 0) ||
 (ns_len == sizeof(SOAP_1_2_ENC_NAMESPACE)-1 
  memcmp(ns, SOAP_1_2_ENC_NAMESPACE, 
sizeof(SOAP_1_2_ENC_NAMESPACE)-1) == 0))) {
-   ns_len = sizeof(XSD_NAMESPACE)-1;
-   len = ns_len + type_len + 1;
-   nscat = emalloc(len + 1);
-   memcpy(nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
-   nscat[ns_len] = ':';
-   memcpy(nscat+ns_len+1, type, type_len);
-   nscat[len] = '\0';
-
-   enc = get_encoder_ex(sdl, nscat, len);
-   efree(nscat);
+   char *enc_nscat;
+   int enc_ns_len;
+   int enc_len;
+
+   enc_ns_len = sizeof(XSD_NAMESPACE)-1;
+   enc_len = enc_ns_len + type_len + 1;
+   enc_nscat = emalloc(enc_len + 1);
+   memcpy(enc_nscat, XSD_NAMESPACE, sizeof(XSD_NAMESPACE)-1);
+   enc_nscat[enc_ns_len] = ':';
+   memcpy(enc_nscat+enc_ns_len+1, type, type_len);
+   enc_nscat[enc_len] = '\0';
+
+   enc = get_encoder_ex(NULL, enc_nscat, enc_len);
+   efree(enc_nscat);
+   if (enc  sdl) {
+   encodePtr new_enc   = emalloc(sizeof(encode));
+   memcpy(new_enc, enc, sizeof(encode));
+   new_enc-details.ns = estrndup(ns, ns_len);
+   new_enc-details.type_str = 
estrdup(new_enc-details.type_str);
+   if (sdl-encoders == NULL) {
+   sdl-encoders = emalloc(sizeof(HashTable));
+   zend_hash_init(sdl-encoders, 0, NULL, 
delete_encoder, 0);
+   }
+   zend_hash_update(sdl-encoders, nscat, len + 1, 
new_enc, sizeof(encodePtr), NULL);
+   enc = new_enc;
+   }
}
+   efree(nscat);
return enc;
 }
 

http://cvs.php.net/co.php/php-src/ext/soap/tests/bugs/bug35273.phpt?r=1.1p=1
Index: php-src/ext/soap/tests/bugs/bug35273.phpt
+++ php-src/ext/soap/tests/bugs/bug35273.phpt
--TEST--
Bug #35273 Error in mapping soap - java types 
--SKIPIF--
?php require_once('skipif.inc'); ?
--FILE--
?php
class TestSoapClient extends SoapClient {
  function __doRequest($request, $location, $action, $version) {
echo $request;
exit;
}
}

ini_set(soap.wsdl_cache_enabled, 0);
$client = new TestSoapClient(dirname(__FILE__).'/bug32941.wsdl', array(trace 
= 1, 'exceptions' = 0));
$ahoj = $client-echoPerson(array(name=Name,surname=Surname));
echo ok\n;
?
--EXPECT--
?xml version=1.0 encoding=UTF-8?
SOAP-ENV:Envelope xmlns:SOAP-ENV=http://schemas.xmlsoap.org/soap/envelope/; 
xmlns:ns1=http://service; 
xmlns:SOAP-ENC=http://schemas.xmlsoap.org/soap/encoding/; 
xmlns:ns2=urn:service.EchoService 
xmlns:xsd=http://www.w3.org/2001/XMLSchema; 
xmlns:xsi=http://www.w3.org/2001/XMLSchema-instance; 

[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS configure.in /main php_version.h

2005-11-18 Thread Derick Rethans
derick  Fri Nov 18 06:07:31 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcconfigure.in NEWS 
/php-src/main   php_version.h 
  Log:
  - Back to -dev.
  
  
http://cvs.php.net/diff.php/php-src/configure.in?r1=1.396.2.164.2.15r2=1.396.2.164.2.16ty=u
Index: php-src/configure.in
diff -u php-src/configure.in:1.396.2.164.2.15 
php-src/configure.in:1.396.2.164.2.16
--- php-src/configure.in:1.396.2.164.2.15   Fri Nov 18 05:30:40 2005
+++ php-src/configure.inFri Nov 18 06:07:25 2005
@@ -1,4 +1,4 @@
-dnl ## $Id: configure.in,v 1.396.2.164.2.15 2005/11/18 10:30:40 derick Exp $ 
-*- sh -*-
+dnl ## $Id: configure.in,v 1.396.2.164.2.16 2005/11/18 11:07:25 derick Exp $ 
-*- sh -*-
 dnl ## Process this file with autoconf to produce a configure script.
 
 divert(1)
@@ -41,7 +41,7 @@
 MAJOR_VERSION=4
 MINOR_VERSION=4
 RELEASE_VERSION=2
-EXTRA_VERSION=RC1
+EXTRA_VERSION=RC2-dev
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$RELEASE_VERSION$EXTRA_VERSION
 
 dnl Define where extension directories are located in the configure context
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.81r2=1.1247.2.920.2.82ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.81 php-src/NEWS:1.1247.2.920.2.82
--- php-src/NEWS:1.1247.2.920.2.81  Fri Nov 18 05:30:37 2005
+++ php-src/NEWSFri Nov 18 06:07:26 2005
@@ -1,5 +1,8 @@
 PHP 4  NEWS
 |||
+23 Nov 2005, Version 4.4.2
+
+
 18 Nov 2005, Version 4.4.2RC1
 - Resolve Apache 2 regression with sub-request handling on non-linux systems.
   (Ilia, Tony)
http://cvs.php.net/diff.php/php-src/main/php_version.h?r1=1.66.2.81.2.12r2=1.66.2.81.2.13ty=u
Index: php-src/main/php_version.h
diff -u php-src/main/php_version.h:1.66.2.81.2.12 
php-src/main/php_version.h:1.66.2.81.2.13
--- php-src/main/php_version.h:1.66.2.81.2.12   Fri Nov 18 05:30:45 2005
+++ php-src/main/php_version.h  Fri Nov 18 06:07:30 2005
@@ -3,5 +3,5 @@
 #define PHP_MAJOR_VERSION 4
 #define PHP_MINOR_VERSION 4
 #define PHP_RELEASE_VERSION 2
-#define PHP_EXTRA_VERSION RC1
-#define PHP_VERSION 4.4.2RC1
+#define PHP_EXTRA_VERSION RC2-dev
+#define PHP_VERSION 4.4.2RC2-dev

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Jani Taskinen

On Fri, 18 Nov 2005, Derick Rethans wrote:



On Thu, 17 Nov 2005, Marcus Boerger wrote:


helly   Thu Nov 17 18:21:15 2005 EDT

  Modified files:
/php-src/ext/reflection config.m4
  Log:
  - Need to define HAVE_REFLECTION


I think it should not be possible to disable this extension with a
simple configure switch...


Why not?

--Jani

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 08:11:41 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
  Log:
  fix news
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.82r2=1.1247.2.920.2.83ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.82 php-src/NEWS:1.1247.2.920.2.83
--- php-src/NEWS:1.1247.2.920.2.82  Fri Nov 18 06:07:26 2005
+++ php-src/NEWSFri Nov 18 08:11:38 2005
@@ -4,10 +4,10 @@
 
 
 18 Nov 2005, Version 4.4.2RC1
-- Resolve Apache 2 regression with sub-request handling on non-linux systems.
+- Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
+- Backported missing imap_mailcompose() fixes from PHP 5.x. (Ilia)
+- Fixed Apache 2 regression with sub-request handling on non-linux systems.
   (Ilia, Tony)
-- Backport missing imap_mailcompose fixes from 5.X. (Ilia)
-- Added missing safe_mode/open_basedir checks in cURL extension. (Ilia)
 - Fixed bug #35257 (Calling ob_flush after creating an ob callback causes 
   segfault). (Tony)
 - Fixed bug #35079 (stream_set_blocking(true) toggles, not enables 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Antony Dovgal

On 18.11.2005 16:08, Jani Taskinen wrote:

On Fri, 18 Nov 2005, Derick Rethans wrote:



On Thu, 17 Nov 2005, Marcus Boerger wrote:


helly   Thu Nov 17 18:21:15 2005 EDT

  Modified files:
/php-src/ext/reflection config.m4
  Log:
  - Need to define HAVE_REFLECTION


I think it should not be possible to disable this extension with a
simple configure switch...


 Why not?


Not that I care, but if reflection could be disabled with --disable-all, 
FreeBSD users will never be able to use it, as for some reason which I can't understand, 
maintainer of PHP port uses --disable-all and builds all extensions as separate packages.

I'm sure also there are a lot of others who use --disable-all and enable only 
what they consider useful.

--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Jani Taskinen

On Fri, 18 Nov 2005, Antony Dovgal wrote:



On 18.11.2005 16:08, Jani Taskinen wrote:

On Fri, 18 Nov 2005, Derick Rethans wrote:



On Thu, 17 Nov 2005, Marcus Boerger wrote:


helly   Thu Nov 17 18:21:15 2005 EDT

  Modified files:
/php-src/ext/reflection config.m4
  Log:
  - Need to define HAVE_REFLECTION


I think it should not be possible to disable this extension with a
simple configure switch...


 Why not?


Not that I care, but if reflection could be disabled with --disable-all, 
FreeBSD users will never be able to use it, as for some reason which I can't 
understand, maintainer of PHP port uses --disable-all and builds all 
extensions as separate packages.
I'm sure also there are a lot of others who use --disable-all and enable only 
what they consider useful.


That's the point of --disable-all and moving this thing to be an extension
was just to make it less broken.

--Jani

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) /ext/readline config.m4

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 10:07:45 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-src/ext/readline   config.m4 
  Log:
  - Make sure --with-readline is used only when --with-libedit is not used.
  
http://cvs.php.net/diff.php/php-src/ext/readline/config.m4?r1=1.25r2=1.25.2.1ty=u
Index: php-src/ext/readline/config.m4
diff -u php-src/ext/readline/config.m4:1.25 
php-src/ext/readline/config.m4:1.25.2.1
--- php-src/ext/readline/config.m4:1.25 Sun May 29 19:16:44 2005
+++ php-src/ext/readline/config.m4  Fri Nov 18 10:07:39 2005
@@ -1,12 +1,14 @@
 dnl
-dnl $Id: config.m4,v 1.25 2005/05/29 23:16:44 sniper Exp $
+dnl $Id: config.m4,v 1.25.2.1 2005/11/18 15:07:39 sniper Exp $
 dnl
 
 PHP_ARG_WITH(libedit,for libedit readline replacement, 
 [  --with-libedit[=DIR]Include libedit readline replacement (CLI/CGI 
only)])
 
-PHP_ARG_WITH(readline,for readline support,
-[  --with-readline[=DIR]   Include readline support (CLI/CGI only)])
+if test $PHP_LIBEDIT = no; then
+  PHP_ARG_WITH(readline,for readline support,
+  [  --with-readline[=DIR]   Include readline support (CLI/CGI only)])
+fi
 
 if test $PHP_READLINE != no; then
   for i in $PHP_READLINE /usr/local /usr; do

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Derick Rethans
On Fri, 18 Nov 2005, Antony Dovgal wrote:

 Not that I care, but if reflection could be disabled with --disable-all,
 FreeBSD users will never be able to use it, as for some reason which I can't
 understand, maintainer of PHP port uses --disable-all and builds all
 extensions as separate packages.

If distributions screw up their packaging it is their fault - not ours.

Derick

-- 
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) /ext/readline config.m4

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 10:10:34 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-src/ext/readline   config.m4 
  Log:
  MF51: sync all changes
  
http://cvs.php.net/diff.php/php-src/ext/readline/config.m4?r1=1.17.4.3r2=1.17.4.3.4.1ty=u
Index: php-src/ext/readline/config.m4
diff -u php-src/ext/readline/config.m4:1.17.4.3 
php-src/ext/readline/config.m4:1.17.4.3.4.1
--- php-src/ext/readline/config.m4:1.17.4.3 Tue Sep 30 22:54:04 2003
+++ php-src/ext/readline/config.m4  Fri Nov 18 10:10:33 2005
@@ -1,12 +1,14 @@
 dnl
-dnl $Id: config.m4,v 1.17.4.3 2003/10/01 02:54:04 sniper Exp $
+dnl $Id: config.m4,v 1.17.4.3.4.1 2005/11/18 15:10:33 sniper Exp $
 dnl
 
 PHP_ARG_WITH(libedit,for libedit readline replacement, 
-[  --with-libedit[=DIR]Include libedit readline replacement (CLI/CGI 
only).])
+[  --with-libedit[=DIR]Include libedit readline replacement (CLI/CGI 
only)])
 
-PHP_ARG_WITH(readline,for readline support,
-[  --with-readline[=DIR]   Include readline support (CLI/CGI only).])
+if test $PHP_LIBEDIT = no; then
+  PHP_ARG_WITH(readline,for readline support,
+  [  --with-readline[=DIR]   Include readline support (CLI/CGI only)])
+fi
 
 if test $PHP_READLINE != no; then
   for i in $PHP_READLINE /usr/local /usr; do
@@ -19,13 +21,16 @@
 
   PHP_ADD_INCLUDE($READLINE_DIR/include)
 
+  PHP_READLINE_LIBS=
   AC_CHECK_LIB(ncurses, tgetent,
   [
 PHP_ADD_LIBRARY(ncurses,,READLINE_SHARED_LIBADD)
+PHP_READLINE_LIBS=$PHP_READLINE_LIBS -lncurses
   ],[
 AC_CHECK_LIB(termcap, tgetent,
 [
   PHP_ADD_LIBRARY(termcap,,READLINE_SHARED_LIBADD)
+  PHP_READLINE_LIBS=$PHP_READLINE_LIBS -ltermcap
 ])
   ])
 
@@ -35,7 +40,14 @@
   ], [
 AC_MSG_ERROR(readline library not found)
   ], [
--L$READLINE_DIR/lib 
+-L$READLINE_DIR/lib $PHP_READLINE_LIBS
+  ])
+
+  PHP_CHECK_LIBRARY(readline, rl_callback_read_char,
+  [
+AC_DEFINE(HAVE_RL_CALLBACK_READ_CHAR, 1, [ ])
+  ],[],[
+-L$READLINE_DIR/lib $PHP_READLINE_LIBS
   ])
 
   PHP_CHECK_LIBRARY(history, add_history,
@@ -44,7 +56,7 @@
   ], [
 AC_MSG_ERROR(history library required by readline not found)
   ], [
--L$READLINE_DIR/lib 
+-L$READLINE_DIR/lib $PHP_READLINE_LIBS
   ])
 
   PHP_NEW_EXTENSION(readline, readline.c, $ext_shared, cli)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src /ext/readline config.m4

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 10:11:14 2005 EDT

  Modified files:  
/php-src/ext/readline   config.m4 
  Log:
  MFB51: fix build when --with-libedit is used
  
http://cvs.php.net/diff.php/php-src/ext/readline/config.m4?r1=1.25r2=1.26ty=u
Index: php-src/ext/readline/config.m4
diff -u php-src/ext/readline/config.m4:1.25 php-src/ext/readline/config.m4:1.26
--- php-src/ext/readline/config.m4:1.25 Sun May 29 19:16:44 2005
+++ php-src/ext/readline/config.m4  Fri Nov 18 10:11:14 2005
@@ -1,12 +1,14 @@
 dnl
-dnl $Id: config.m4,v 1.25 2005/05/29 23:16:44 sniper Exp $
+dnl $Id: config.m4,v 1.26 2005/11/18 15:11:14 sniper Exp $
 dnl
 
 PHP_ARG_WITH(libedit,for libedit readline replacement, 
 [  --with-libedit[=DIR]Include libedit readline replacement (CLI/CGI 
only)])
 
-PHP_ARG_WITH(readline,for readline support,
-[  --with-readline[=DIR]   Include readline support (CLI/CGI only)])
+if test $PHP_LIBEDIT = no; then
+  PHP_ARG_WITH(readline,for readline support,
+  [  --with-readline[=DIR]   Include readline support (CLI/CGI only)])
+fi
 
 if test $PHP_READLINE != no; then
   for i in $PHP_READLINE /usr/local /usr; do

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Antony Dovgal

On 18.11.2005 18:03, Jani Taskinen wrote:
Not that I care, but if reflection could be disabled with --disable-all, 
FreeBSD users will never be able to use it, as for some reason which I can't 
understand, maintainer of PHP port uses --disable-all and builds all 
extensions as separate packages.
I'm sure also there are a lot of others who use --disable-all and enable only 
what they consider useful.


 That's the point of --disable-all and moving this thing to be an extension
 was just to make it less broken.


Probably, I explained it not very good.
ext/reflection cannot be built as shared and that'll prevent FreeBSD users to 
use it at all.
That's why I don't like the idea of having such an easy way to disable it.

--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Antony Dovgal

On 18.11.2005 18:08, Derick Rethans wrote:

On Fri, 18 Nov 2005, Antony Dovgal wrote:


Not that I care, but if reflection could be disabled with --disable-all,
FreeBSD users will never be able to use it, as for some reason which I can't
understand, maintainer of PHP port uses --disable-all and builds all
extensions as separate packages.


If distributions screw up their packaging it is their fault - not ours.


Exactly.
But we can think of making PHP a bit harder to screw up, right?

--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Derick Rethans
On Fri, 18 Nov 2005, Antony Dovgal wrote:

 On 18.11.2005 18:08, Derick Rethans wrote:
  On Fri, 18 Nov 2005, Antony Dovgal wrote:
  
   Not that I care, but if reflection could be disabled with --disable-all,
   FreeBSD users will never be able to use it, as for some reason which I
   can't
   understand, maintainer of PHP port uses --disable-all and builds all
   extensions as separate packages.
  
  If distributions screw up their packaging it is their fault - not ours.
 
 Exactly.
 But we can think of making PHP a bit harder to screw up, right?

--disable-all is not meant for distributions, but they are find to shoot 
themselves in the foot. I don't see why we should try to prevent that. 
There is no reason why a distribution should use --disable-all and then 
compile everything shared. There is no reason for that. 

Derick

-- 
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Derick Rethans
On Fri, 18 Nov 2005, Antony Dovgal wrote:

 We need at least document it or to add something like this:

But then we need to do this for every configure option... as nobody 
really knows what they are doing anyway.

Derick

-- 
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Jani Taskinen

On Fri, 18 Nov 2005, Antony Dovgal wrote:



On 18.11.2005 18:03, Jani Taskinen wrote:
Not that I care, but if reflection could be disabled with --disable-all, 
FreeBSD users will never be able to use it, as for some reason which I 
can't understand, maintainer of PHP port uses --disable-all and builds all 
extensions as separate packages.
I'm sure also there are a lot of others who use --disable-all and enable 
only what they consider useful.


 That's the point of --disable-all and moving this thing to be an 
extension

 was just to make it less broken.


Probably, I explained it not very good.
ext/reflection cannot be built as shared and that'll prevent FreeBSD users to 
use it at all.


Doing '--enable-reflection' is not possible on FreeBSD?
And that's our problem, because..?

--Jani

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Jani Taskinen


There's no point in putting any more notes in the configure help.
I've seen how many people read them for real already. None.

--Jani


On Fri, 18 Nov 2005, Antony Dovgal wrote:



On 18.11.2005 18:25, Derick Rethans wrote:

On Fri, 18 Nov 2005, Antony Dovgal wrote:


On 18.11.2005 18:08, Derick Rethans wrote:
 On Fri, 18 Nov 2005, Antony Dovgal wrote:
   Not that I care, but if reflection could be disabled with 
--disable-all,

  FreeBSD users will never be able to use it, as for some reason which I
  can't
  understand, maintainer of PHP port uses --disable-all and builds all
  extensions as separate packages.
  If distributions screw up their packaging it is their fault - not ours.

Exactly.
But we can think of making PHP a bit harder to screw up, right?


--disable-all is not meant for distributions, but they are find to shoot 
themselves in the foot. I don't see why we should try to prevent that. There 
is no reason why a distribution should use --disable-all and then compile 
everything shared. There is no reason for that. 


I do know that.
But *maintainers* often don't.

We need at least document it or to add something like this:

Index: configure.in
===
RCS file: /repository/php-src/configure.in,v
retrieving revision 1.579.2.18
diff -u -p -d -r1.579.2.18 configure.in
--- configure.in17 Nov 2005 21:27:56 -  1.579.2.18
+++ configure.in18 Nov 2005 15:36:43 -
@@ -790,7 +790,10 @@ dnl Check if all enabled by default exte
dnl

AC_ARG_ENABLE(all,
-[ --disable-all   Disable all extensions which are enabled by default
+[ --disable-all   Disable all extensions which are enabled by default.
+  ONLY DO THAT IF KNOW WHAT YOU'RE DOING!
+  All default extensions have a good reason to be
+  enabled by default!
], [
 PHP_ENABLE_ALL=$enableval
])




--
Give me your money at @ http://pecl.php.net/wishlist.php/sniper
Donating money may make me happier and friendlier for a limited period!
Death to all 4 letter abbreviations starting with P!

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / INSTALL

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 11:09:51 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcINSTALL 
  Log:
  php4 - php5
  
http://cvs.php.net/diff.php/php-src/INSTALL?r1=1.37r2=1.37.2.1ty=u
Index: php-src/INSTALL
diff -u php-src/INSTALL:1.37 php-src/INSTALL:1.37.2.1
--- php-src/INSTALL:1.37Wed Mar 30 08:15:05 2005
+++ php-src/INSTALL Fri Nov 18 11:09:50 2005
@@ -262,8 +262,8 @@
 
 10. cd ../apache_1.3.x
 
-11. ./configure --prefix=/www --activate-module=src/modules/php4/libphp4.a
-(The above line is correct! Yes, we know libphp4.a does not exist at this
+11. ./configure --prefix=/www --activate-module=src/modules/php5/libphp5.a
+(The above line is correct! Yes, we know libphp5.a does not exist at this
 stage. It isn't supposed to. It will be created.)
 
 12. make
@@ -309,24 +309,24 @@
 
./configure --with-apxs --with-pgsql
 
-   This will create a libphp4.so shared library that is loaded into
+   This will create a libphp5.so shared library that is loaded into
Apache using a LoadModule line in Apache's httpd.conf file. The
-   PostgreSQL support is embedded into this libphp4.so library.
+   PostgreSQL support is embedded into this libphp5.so library.
 
./configure --with-apxs --with-pgsql=shared
 
-   This will create a libphp4.so shared library for Apache, but it will
+   This will create a libphp5.so shared library for Apache, but it will
also create a pgsql.so shared library that is loaded into PHP either
by using the extension directive in php.ini file or by loading it
explicitly in a script using the dl() function.
 
./configure --with-apache=/path/to/apache_source --with-pgsql
 
-   This will create a libmodphp4.a library, a mod_php4.c and some
-   accompanying files and copy this into the src/modules/php4 directory
+   This will create a libmodphp5.a library, a mod_php5.c and some
+   accompanying files and copy this into the src/modules/php5 directory
in the Apache source tree. Then you compile Apache using
-   --activate-module=src/modules/php4/libphp4.a and the Apache build
-   system will create libphp4.a and link it statically into the httpd
+   --activate-module=src/modules/php5/libphp5.a and the Apache build
+   system will create libphp5.a and link it statically into the httpd
binary. The PostgreSQL support is included directly into this httpd
binary, so the final result here is a single httpd binary that
includes all of Apache and all of PHP.
@@ -380,8 +380,8 @@
 
  * PHP 4.3.0 or later available at http://www.php.net/downloads.php.
  * the latest stable development version. Get the source code
-   http://snaps.php.net/php4-latest.tar.gz or download binaries for
-   Windows http://snaps.php.net/win32/php4-win32-latest.zip.
+   http://snaps.php.net/php5-latest.tar.gz or download binaries for
+   Windows http://snaps.php.net/win32/php5-win32-latest.zip.
  * a prerelease version downloadable from http://qa.php.net/.
  * you have always the option to obtain PHP through anonymous CVS.
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / php.ini-dist php.ini-recommended /main main.c php_globals.h /main/streams streams.c

2005-11-18 Thread Rasmus Lerdorf
rasmus  Fri Nov 18 11:20:43 2005 EDT

  Modified files:  
/php-srcphp.ini-dist php.ini-recommended 
/php-src/main   main.c php_globals.h 
/php-src/main/streams   streams.c 
  Log:
  Add allow_url_include to let people turn on allow_url_fopen without also
  enabling remote code execution through url wrappers
  
  
http://cvs.php.net/diff.php/php-src/php.ini-dist?r1=1.237r2=1.238ty=u
Index: php-src/php.ini-dist
diff -u php-src/php.ini-dist:1.237 php-src/php.ini-dist:1.238
--- php-src/php.ini-dist:1.237  Mon Nov 14 18:14:54 2005
+++ php-src/php.ini-distFri Nov 18 11:20:41 2005
@@ -529,6 +529,8 @@
 
 ; Whether to allow the treatment of URLs (like http:// or ftp://) as files.
 allow_url_fopen = On
+; Whether to allow code execution through URL wrappers
+allow_url_include = Off
 
 ; Define the anonymous ftp password (your email address)
 ;from=[EMAIL PROTECTED]
http://cvs.php.net/diff.php/php-src/php.ini-recommended?r1=1.186r2=1.187ty=u
Index: php-src/php.ini-recommended
diff -u php-src/php.ini-recommended:1.186 php-src/php.ini-recommended:1.187
--- php-src/php.ini-recommended:1.186   Mon Nov 14 18:14:54 2005
+++ php-src/php.ini-recommended Fri Nov 18 11:20:41 2005
@@ -586,6 +586,8 @@
 
 ; Whether to allow the treatment of URLs (like http:// or ftp://) as files.
 allow_url_fopen = On
+; Whether to allow code execution through URL wrappers
+allow_url_include = Off
 
 ; Define the anonymous ftp password (your email address)
 ;from=[EMAIL PROTECTED]
http://cvs.php.net/diff.php/php-src/main/main.c?r1=1.651r2=1.652ty=u
Index: php-src/main/main.c
diff -u php-src/main/main.c:1.651 php-src/main/main.c:1.652
--- php-src/main/main.c:1.651   Tue Nov 15 06:14:34 2005
+++ php-src/main/main.c Fri Nov 18 11:20:43 2005
@@ -18,7 +18,7 @@
+--+
 */
 
-/* $Id: main.c,v 1.651 2005/11/15 11:14:34 dmitry Exp $ */
+/* $Id: main.c,v 1.652 2005/11/18 16:20:43 rasmus Exp $ */
 
 /* {{{ includes
  */
@@ -439,6 +439,7 @@
PHP_INI_ENTRY(disable_classes,, 
PHP_INI_SYSTEM, NULL)
 
STD_PHP_INI_BOOLEAN(allow_url_fopen,  1,
PHP_INI_SYSTEM, OnUpdateBool,   allow_url_fopen,
php_core_globals,   core_globals)
+   STD_PHP_INI_BOOLEAN(allow_url_include,0,
PHP_INI_SYSTEM, OnUpdateBool,   allow_url_include,  
php_core_globals,   core_globals)
STD_PHP_INI_BOOLEAN(always_populate_raw_post_data,0,
PHP_INI_SYSTEM|PHP_INI_PERDIR,  OnUpdateBool,   
always_populate_raw_post_data,  php_core_globals,   
core_globals)
 #ifdef REALPATH_CACHE
STD_PHP_INI_ENTRY(realpath_cache_size, 16K, PHP_INI_SYSTEM, 
OnUpdateLong, realpath_cache_size_limit, virtual_cwd_globals, cwd_globals)
http://cvs.php.net/diff.php/php-src/main/php_globals.h?r1=1.98r2=1.99ty=u
Index: php-src/main/php_globals.h
diff -u php-src/main/php_globals.h:1.98 php-src/main/php_globals.h:1.99
--- php-src/main/php_globals.h:1.98 Wed Aug  3 10:08:33 2005
+++ php-src/main/php_globals.h  Fri Nov 18 11:20:43 2005
@@ -16,7 +16,7 @@
+--+
 */
 
-/* $Id: php_globals.h,v 1.98 2005/08/03 14:08:33 sniper Exp $ */
+/* $Id: php_globals.h,v 1.99 2005/11/18 16:20:43 rasmus Exp $ */
 
 #ifndef PHP_GLOBALS_H
 #define PHP_GLOBALS_H
@@ -150,6 +150,7 @@
 
char *disable_functions;
char *disable_classes;
+   zend_bool allow_url_include;
 };
 
 
http://cvs.php.net/diff.php/php-src/main/streams/streams.c?r1=1.89r2=1.90ty=u
Index: php-src/main/streams/streams.c
diff -u php-src/main/streams/streams.c:1.89 php-src/main/streams/streams.c:1.90
--- php-src/main/streams/streams.c:1.89 Wed Oct  5 17:46:28 2005
+++ php-src/main/streams/streams.c  Fri Nov 18 11:20:43 2005
@@ -19,7 +19,7 @@
+--+
  */
 
-/* $Id: streams.c,v 1.89 2005/10/05 21:46:28 tony2001 Exp $ */
+/* $Id: streams.c,v 1.90 2005/11/18 16:20:43 rasmus Exp $ */
 
 #define _GNU_SOURCE
 #include php.h
@@ -2205,7 +2205,7 @@
return php_plain_files_wrapper;
}
 
-   if (wrapper  wrapper-is_url  !PG(allow_url_fopen)) {
+   if ((wrapper  wrapper-is_url)  (!PG(allow_url_fopen) || (options  
STREAM_OPEN_FOR_INCLUDE)  !PG(allow_url_include)) ) {
if (options  REPORT_ERRORS) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, URL 
file-access is disabled in the server configuration);
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / NEWS

2005-11-18 Thread Jani Taskinen
sniper  Fri Nov 18 11:41:50 2005 EDT

  Modified files:  
/php-srcNEWS 
  Log:
  reorder
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2078r2=1.2079ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2078 php-src/NEWS:1.2079
--- php-src/NEWS:1.2078 Fri Nov 18 11:28:01 2005
+++ php-src/NEWSFri Nov 18 11:41:49 2005
@@ -8,6 +8,8 @@
   (Marcus, Dmitry)
 - Changed instanceof and catch operators, is_a() and is_subclass_of()
   functions to not call __autoload(). (Dmitry)
+- Added allow_url_include ini directive to be able to turn off remote url
+  code execution separately from the allow_url_fopen setting. (Rasmus)
 - Added --enable-gcov configure option to enable C-level code coverage.
   (John, Jani)
 - Added to cURL extension: (Ilia)
@@ -24,5 +26,3 @@
 - Added possibility to check in which extension an internal function was
   defined using reflection API. (Johannes)
 - Fixed bug #34286 (__toString() behavior is inconsistent). (Marcus)
-- Add allow_url_include ini directive to be able to turn off remote url 
-  code execution separately from the allow_url_fopen setting. (Rasmus)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /sapi/apache2handler sapi_apache2.c

2005-11-18 Thread Ilia Alshanetsky
iliaa   Fri Nov 18 13:59:51 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcNEWS 
/php-src/sapi/apache2handlersapi_apache2.c 
  Log:
  Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module).
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.202r2=1.2027.2.203ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.202 php-src/NEWS:1.2027.2.203
--- php-src/NEWS:1.2027.2.202   Fri Nov 18 06:00:14 2005
+++ php-src/NEWSFri Nov 18 13:59:46 2005
@@ -1,6 +1,7 @@
 PHPNEWS
 |||
 ?? Nov 2005, PHP 5.1
+- Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6
http://cvs.php.net/diff.php/php-src/sapi/apache2handler/sapi_apache2.c?r1=1.57.2.6r2=1.57.2.7ty=u
Index: php-src/sapi/apache2handler/sapi_apache2.c
diff -u php-src/sapi/apache2handler/sapi_apache2.c:1.57.2.6 
php-src/sapi/apache2handler/sapi_apache2.c:1.57.2.7
--- php-src/sapi/apache2handler/sapi_apache2.c:1.57.2.6 Thu Nov 17 16:00:28 2005
+++ php-src/sapi/apache2handler/sapi_apache2.c  Fri Nov 18 13:59:49 2005
@@ -18,7 +18,7 @@
+--+
  */
 
-/* $Id: sapi_apache2.c,v 1.57.2.6 2005/11/17 21:00:28 iliaa Exp $ */
+/* $Id: sapi_apache2.c,v 1.57.2.7 2005/11/18 18:59:49 iliaa Exp $ */
 
 #define ZEND_INCLUDE_FULL_WINDOWS_HEADERS
 
@@ -443,6 +443,18 @@
php_request_shutdown(NULL);
 }
 
+static void php_apache_ini_dtor(request_rec *r, request_rec *p TSRMLS_DC)
+{
+   if (strcmp(r-protocol, INCLUDED)) {
+   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try();
+   }
+   if (p) {
+   ((php_struct *)SG(server_context))-r = p;
+   } else {
+   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup);
+   }
+}
+
 static int php_handler(request_rec *r)
 {
php_struct *ctx;
@@ -453,11 +465,7 @@
request_rec *parent_req = NULL;
TSRMLS_FETCH();
 
-#define PHPAP_INI_OFF \
-   if (strcmp(r-protocol, INCLUDED)) { \
-   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try(); \
-   } \
-   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup); \
+#define PHPAP_INI_OFF php_apache_ini_dtor(r, parent_req TSRMLS_CC);
 
conf = ap_get_module_config(r-per_dir_config, php5_module);
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src /sapi/apache2handler sapi_apache2.c

2005-11-18 Thread Ilia Alshanetsky
iliaa   Fri Nov 18 14:01:50 2005 EDT

  Modified files:  
/php-src/sapi/apache2handlersapi_apache2.c 
  Log:
  MFB51: Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module)
  
  
http://cvs.php.net/diff.php/php-src/sapi/apache2handler/sapi_apache2.c?r1=1.63r2=1.64ty=u
Index: php-src/sapi/apache2handler/sapi_apache2.c
diff -u php-src/sapi/apache2handler/sapi_apache2.c:1.63 
php-src/sapi/apache2handler/sapi_apache2.c:1.64
--- php-src/sapi/apache2handler/sapi_apache2.c:1.63 Thu Nov 17 15:56:09 2005
+++ php-src/sapi/apache2handler/sapi_apache2.c  Fri Nov 18 14:01:50 2005
@@ -18,7 +18,7 @@
+--+
  */
 
-/* $Id: sapi_apache2.c,v 1.63 2005/11/17 20:56:09 iliaa Exp $ */
+/* $Id: sapi_apache2.c,v 1.64 2005/11/18 19:01:50 iliaa Exp $ */
 
 #define ZEND_INCLUDE_FULL_WINDOWS_HEADERS
 
@@ -443,6 +443,18 @@
php_request_shutdown(NULL);
 }
 
+static void php_apache_ini_dtor(request_rec *r, request_rec *p TSRMLS_DC)
+{
+   if (strcmp(r-protocol, INCLUDED)) {
+   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try();
+   }
+   if (p) {
+   ((php_struct *)SG(server_context))-r = p;
+   } else {
+   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup);
+   }
+}
+
 static int php_handler(request_rec *r)
 {
php_struct *ctx;
@@ -453,11 +465,7 @@
request_rec *parent_req = NULL;
TSRMLS_FETCH();
 
-#define PHPAP_INI_OFF \
-   if (strcmp(r-protocol, INCLUDED)) { \
-   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try(); \
-   } \
-   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup); \
+#define PHPAP_INI_OFF php_apache_ini_dtor(r, parent_req TSRMLS_CC);
 
conf = ap_get_module_config(r-per_dir_config, php5_module);
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /sapi/apache2handler sapi_apache2.c

2005-11-18 Thread Ilia Alshanetsky
iliaa   Fri Nov 18 14:03:14 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
/php-src/sapi/apache2handlersapi_apache2.c 
  Log:
  MFH: Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module).
  
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.83r2=1.1247.2.920.2.84ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.83 php-src/NEWS:1.1247.2.920.2.84
--- php-src/NEWS:1.1247.2.920.2.83  Fri Nov 18 08:11:38 2005
+++ php-src/NEWSFri Nov 18 14:03:09 2005
@@ -1,7 +1,7 @@
 PHP 4  NEWS
 |||
 23 Nov 2005, Version 4.4.2
-
+- Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 
 18 Nov 2005, Version 4.4.2RC1
 - Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
http://cvs.php.net/diff.php/php-src/sapi/apache2handler/sapi_apache2.c?r1=1.1.2.40.2.7r2=1.1.2.40.2.8ty=u
Index: php-src/sapi/apache2handler/sapi_apache2.c
diff -u php-src/sapi/apache2handler/sapi_apache2.c:1.1.2.40.2.7 
php-src/sapi/apache2handler/sapi_apache2.c:1.1.2.40.2.8
--- php-src/sapi/apache2handler/sapi_apache2.c:1.1.2.40.2.7 Thu Nov 17 
16:01:13 2005
+++ php-src/sapi/apache2handler/sapi_apache2.c  Fri Nov 18 14:03:13 2005
@@ -18,7 +18,7 @@
+--+
  */
 
-/* $Id: sapi_apache2.c,v 1.1.2.40.2.7 2005/11/17 21:01:13 iliaa Exp $ */
+/* $Id: sapi_apache2.c,v 1.1.2.40.2.8 2005/11/18 19:03:13 iliaa Exp $ */
 
 #include fcntl.h
 
@@ -443,6 +443,18 @@
php_request_shutdown(NULL);
 }
 
+static void php_apache_ini_dtor(request_rec *r, request_rec *p TSRMLS_DC)
+{
+   if (strcmp(r-protocol, INCLUDED)) {
+   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try();
+   }
+   if (p) {
+   ((php_struct *)SG(server_context))-r = p;
+   } else {
+   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup);
+   }
+}
+
 static int php_handler(request_rec *r)
 {
php_struct *ctx;
@@ -453,11 +465,7 @@
request_rec *parent_req = NULL;
TSRMLS_FETCH();
 
-#define PHPAP_INI_OFF \
-   if (strcmp(r-protocol, INCLUDED)) { \
-   zend_try { zend_ini_deactivate(TSRMLS_C); } zend_end_try(); \
-   } \
-   apr_pool_cleanup_run(r-pool, (void *)SG(server_context), 
php_server_context_cleanup); \
+#define PHPAP_INI_OFF php_apache_ini_dtor(r, parent_req TSRMLS_CC);
 
conf = ap_get_module_config(r-per_dir_config, php4_module);
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 14:13:36 2005 EDT

  Modified files:  
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2079r2=1.2080ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2079 php-src/NEWS:1.2080
--- php-src/NEWS:1.2079 Fri Nov 18 11:41:49 2005
+++ php-src/NEWSFri Nov 18 14:13:34 2005
@@ -26,3 +26,4 @@
 - Added possibility to check in which extension an internal function was
   defined using reflection API. (Johannes)
 - Fixed bug #34286 (__toString() behavior is inconsistent). (Marcus)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.155r2=1.156ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.155 php-src/ext/mssql/php_mssql.c:1.156
--- php-src/ext/mssql/php_mssql.c:1.155 Mon Nov 14 18:14:54 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:13:35 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.155 2005/11/14 23:14:54 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.156 2005/11/18 19:13:35 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -1865,6 +1865,11 @@
 
mssql_ptr = result-mssql_ptr;
retvalue = dbresults(mssql_ptr-link);
+   
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 14:14:33 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.203r2=1.2027.2.204ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.203 php-src/NEWS:1.2027.2.204
--- php-src/NEWS:1.2027.2.203   Fri Nov 18 13:59:46 2005
+++ php-src/NEWSFri Nov 18 14:14:31 2005
@@ -3,6 +3,7 @@
 ?? Nov 2005, PHP 5.1
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6
 - Changed function parameter parsing to handle integers in a non-strict fashion
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.152.2.4r2=1.152.2.5ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.152.2.4 
php-src/ext/mssql/php_mssql.c:1.152.2.5
--- php-src/ext/mssql/php_mssql.c:1.152.2.4 Mon Nov 14 18:14:25 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:14:32 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.152.2.4 2005/11/14 23:14:25 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.152.2.5 2005/11/18 19:14:32 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -1865,6 +1865,11 @@
 
mssql_ptr = result-mssql_ptr;
retvalue = dbresults(mssql_ptr-link);
+
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 14:15:14 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.84r2=1.1247.2.920.2.85ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.84 php-src/NEWS:1.1247.2.920.2.85
--- php-src/NEWS:1.1247.2.920.2.84  Fri Nov 18 14:03:09 2005
+++ php-src/NEWSFri Nov 18 14:15:11 2005
@@ -2,6 +2,7 @@
 |||
 23 Nov 2005, Version 4.4.2
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
 
 18 Nov 2005, Version 4.4.2RC1
 - Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.86.2.44.2.4r2=1.86.2.44.2.5ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.86.2.44.2.4 
php-src/ext/mssql/php_mssql.c:1.86.2.44.2.5
--- php-src/ext/mssql/php_mssql.c:1.86.2.44.2.4 Tue Nov 15 12:33:07 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:15:12 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.86.2.44.2.4 2005/11/15 17:33:07 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.86.2.44.2.5 2005/11/18 19:15:12 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -1874,6 +1874,11 @@
 
mssql_ptr = result-mssql_ptr;
retvalue = dbresults(mssql_ptr-link);
+
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Antony Dovgal

Please, MFH to PHP_5_0 too.
Thanks in advance.

On 18.11.2005 22:14, Frank M. Kromann wrote:

fmk Fri Nov 18 14:14:33 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-src	NEWS 
/php-src/ext/mssql	php_mssql.c 
  Log:

  MFH: Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.203r2=1.2027.2.204ty=u

Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.203 php-src/NEWS:1.2027.2.204
--- php-src/NEWS:1.2027.2.203   Fri Nov 18 13:59:46 2005
+++ php-src/NEWSFri Nov 18 14:14:31 2005
@@ -3,6 +3,7 @@
 ?? Nov 2005, PHP 5.1
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6

 - Changed function parameter parsing to handle integers in a non-strict fashion
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.152.2.4r2=1.152.2.5ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.152.2.4 
php-src/ext/mssql/php_mssql.c:1.152.2.5
--- php-src/ext/mssql/php_mssql.c:1.152.2.4 Mon Nov 14 18:14:25 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:14:32 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.152.2.4 2005/11/14 23:14:25 fmk Exp $ */

+/* $Id: php_mssql.c,v 1.152.2.5 2005/11/18 19:14:32 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL

 #define HAVE_MSSQL 1
@@ -1865,6 +1865,11 @@
 
 	mssql_ptr = result-mssql_ptr;

retvalue = dbresults(mssql_ptr-link);
+
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}




--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 14:43:16 2005 EDT

  Modified files:  (Branch: PHP_5_0)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.516r2=1.1760.2.517ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.516 php-src/NEWS:1.1760.2.517
--- php-src/NEWS:1.1760.2.516   Fri Nov 18 06:01:04 2005
+++ php-src/NEWSFri Nov 18 14:43:03 2005
@@ -58,6 +58,7 @@
 - Fixed bug #33383 (crash when retrieving empty LOBs). (Tony)
 - Fixed bug #32179 (xmlrpc_encode() segfaults with recursive references). 
(Tony)
 - Fixed bug #29983 (PHP does not explicitly set mime type  charset). (Ilia)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
 
 05 Sep 2005, PHP 5.0.5
 - Upgraded PCRE library to version 5.0. (Andrei)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.137.2.11r2=1.137.2.12ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.137.2.11 
php-src/ext/mssql/php_mssql.c:1.137.2.12
--- php-src/ext/mssql/php_mssql.c:1.137.2.11Wed Nov 16 13:24:30 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:43:15 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.137.2.11 2005/11/16 18:24:30 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.137.2.12 2005/11/18 19:43:15 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -1835,6 +1835,11 @@
 
mssql_ptr = result-mssql_ptr;
retvalue = dbresults(mssql_ptr-link);
+
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Antony Dovgal

Thanks again.

On 18.11.2005 22:43, Frank M. Kromann wrote:

fmk Fri Nov 18 14:43:16 2005 EDT

  Modified files:  (Branch: PHP_5_0)
/php-src	NEWS 
/php-src/ext/mssql	php_mssql.c 
  Log:

  MFH: Fix #33153 Crash in mssql_next_result().
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.516r2=1.1760.2.517ty=u

Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.516 php-src/NEWS:1.1760.2.517
--- php-src/NEWS:1.1760.2.516   Fri Nov 18 06:01:04 2005
+++ php-src/NEWSFri Nov 18 14:43:03 2005
@@ -58,6 +58,7 @@
 - Fixed bug #33383 (crash when retrieving empty LOBs). (Tony)
 - Fixed bug #32179 (xmlrpc_encode() segfaults with recursive references). 
(Tony)
 - Fixed bug #29983 (PHP does not explicitly set mime type  charset). (Ilia)
+- Fixed bug #33153 (crash in mssql_next result). (Frank)
 
 05 Sep 2005, PHP 5.0.5

 - Upgraded PCRE library to version 5.0. (Andrei)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.137.2.11r2=1.137.2.12ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.137.2.11 
php-src/ext/mssql/php_mssql.c:1.137.2.12
--- php-src/ext/mssql/php_mssql.c:1.137.2.11Wed Nov 16 13:24:30 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 14:43:15 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.137.2.11 2005/11/16 18:24:30 fmk Exp $ */

+/* $Id: php_mssql.c,v 1.137.2.12 2005/11/18 19:43:15 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL

 #define HAVE_MSSQL 1
@@ -1835,6 +1835,11 @@
 
 	mssql_ptr = result-mssql_ptr;

retvalue = dbresults(mssql_ptr-link);
+
+   while (dbnumcols(mssql_ptr-link) = 0  retvalue == SUCCEED) {
+   retvalue = dbresults(mssql_ptr-link);
+   }
+
if (retvalue == FAIL) {
RETURN_FALSE;
}




--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src(PHP_5_1) / INSTALL

2005-11-18 Thread Nuno Lopes
Please don't touch in the INSTALL and win32/install.txt files directly, 
because they are created automatically from the phpdoc files (make 
install-unix.txt  make install-win.txt)


Thanks,
Nuno


- Original Message - 

sniper Fri Nov 18 11:09:51 2005 EDT

 Modified files:  (Branch: PHP_5_1)
   /php-src INSTALL
 Log:
 php4 - php5

http://cvs.php.net/diff.php/php-src/INSTALL?r1=1.37r2=1.37.2.1ty=u
Index: php-src/INSTALL
diff -u php-src/INSTALL:1.37 php-src/INSTALL:1.37.2.1
--- php-src/INSTALL:1.37 Wed Mar 30 08:15:05 2005
+++ php-src/INSTALL Fri Nov 18 11:09:50 2005
@@ -262,8 +262,8 @@

10. cd ../apache_1.3.x

-11. 
./configure --prefix=/www --activate-module=src/modules/php4/libphp4.a
-(The above line is correct! Yes, we know libphp4.a does not exist at 
this
+11. 
./configure --prefix=/www --activate-module=src/modules/php5/libphp5.a
+(The above line is correct! Yes, we know libphp5.a does not exist at 
this

stage. It isn't supposed to. It will be created.)

12. make
@@ -309,24 +309,24 @@

   ./configure --with-apxs --with-pgsql

-   This will create a libphp4.so shared library that is loaded into
+   This will create a libphp5.so shared library that is loaded into
   Apache using a LoadModule line in Apache's httpd.conf file. The
-   PostgreSQL support is embedded into this libphp4.so library.
+   PostgreSQL support is embedded into this libphp5.so library.

   ./configure --with-apxs --with-pgsql=shared

-   This will create a libphp4.so shared library for Apache, but it will
+   This will create a libphp5.so shared library for Apache, but it will
   also create a pgsql.so shared library that is loaded into PHP either
   by using the extension directive in php.ini file or by loading it
   explicitly in a script using the dl() function.

   ./configure --with-apache=/path/to/apache_source --with-pgsql

-   This will create a libmodphp4.a library, a mod_php4.c and some
-   accompanying files and copy this into the src/modules/php4 directory
+   This will create a libmodphp5.a library, a mod_php5.c and some
+   accompanying files and copy this into the src/modules/php5 directory
   in the Apache source tree. Then you compile Apache using
-   --activate-module=src/modules/php4/libphp4.a and the Apache build
-   system will create libphp4.a and link it statically into the httpd
+   --activate-module=src/modules/php5/libphp5.a and the Apache build
+   system will create libphp5.a and link it statically into the httpd
   binary. The PostgreSQL support is included directly into this httpd
   binary, so the final result here is a single httpd binary that
   includes all of Apache and all of PHP.
@@ -380,8 +380,8 @@

 * PHP 4.3.0 or later available at http://www.php.net/downloads.php.
 * the latest stable development version. Get the source code
-   http://snaps.php.net/php4-latest.tar.gz or download binaries for
-   Windows http://snaps.php.net/win32/php4-win32-latest.zip.
+   http://snaps.php.net/php5-latest.tar.gz or download binaries for
+   Windows http://snaps.php.net/win32/php5-win32-latest.zip.
 * a prerelease version downloadable from http://qa.php.net/.
 * you have always the option to obtain PHP through anonymous CVS. 


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:40:28 2005 EDT

  Modified files:  
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  Fix #32009 crash when mssql_bind() is called more than once
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2080r2=1.2081ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2080 php-src/NEWS:1.2081
--- php-src/NEWS:1.2080 Fri Nov 18 14:13:34 2005
+++ php-src/NEWSFri Nov 18 15:40:19 2005
@@ -27,3 +27,4 @@
   defined using reflection API. (Johannes)
 - Fixed bug #34286 (__toString() behavior is inconsistent). (Marcus)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
+- Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.156r2=1.157ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.156 php-src/ext/mssql/php_mssql.c:1.157
--- php-src/ext/mssql/php_mssql.c:1.156 Fri Nov 18 14:13:35 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:40:28 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.156 2005/11/18 19:13:35 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.157 2005/11/18 20:40:28 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2114,17 +2114,22 @@
zend_hash_init(statement-binds, 13, NULL, 
_mssql_bind_hash_dtor, 0);
}
 
-   memset((void*)bind,0,sizeof(mssql_bind));
-   
zend_hash_add(statement-binds,Z_STRVAL_PP(param_name),Z_STRLEN_PP(param_name),bind,sizeof(mssql_bind),(void
 **)bindp);
-   if( NULL == bindp ) RETURN_FALSE;
-   bindp-zval=*var;
-   zval_add_ref(var);
-
-   /* no call to dbrpcparam if RETVAL */
-   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

-   if (dbrpcparam(mssql_ptr-link, Z_STRVAL_PP(param_name), 
(BYTE)status, type, maxlen, datalen, (LPBYTE)value)==FAIL) {
-   php_error_docref(NULL TSRMLS_CC, E_WARNING, Unable to 
set parameter);
-   RETURN_FALSE;
+   if (zend_hash_exists(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name))) {
+   RETURN_FALSE;
+   }
+   else {
+   memset((void*)bind,0,sizeof(mssql_bind));
+   zend_hash_add(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name), bind, sizeof(mssql_bind), (void **)bindp);
+   if( NULL == bindp ) RETURN_FALSE;
+   bindp-zval=*var;
+   zval_add_ref(var);
+   
+   /* no call to dbrpcparam if RETVAL */
+   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

+   if (dbrpcparam(mssql_ptr-link, 
Z_STRVAL_PP(param_name), (BYTE)status, type, maxlen, datalen, 
(LPBYTE)value)==FAIL) {
+   php_error_docref(NULL TSRMLS_CC, E_WARNING, 
Unable to set parameter);
+   RETURN_FALSE;
+   }
}
}
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:40:53 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #32009 crash when mssql_bind() is called more than once
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.204r2=1.2027.2.205ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.204 php-src/NEWS:1.2027.2.205
--- php-src/NEWS:1.2027.2.204   Fri Nov 18 14:14:31 2005
+++ php-src/NEWSFri Nov 18 15:40:51 2005
@@ -4,6 +4,7 @@
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
+- Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6
 - Changed function parameter parsing to handle integers in a non-strict fashion
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.152.2.5r2=1.152.2.6ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.152.2.5 
php-src/ext/mssql/php_mssql.c:1.152.2.6
--- php-src/ext/mssql/php_mssql.c:1.152.2.5 Fri Nov 18 14:14:32 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:40:52 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.152.2.5 2005/11/18 19:14:32 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.152.2.6 2005/11/18 20:40:52 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2116,17 +2116,22 @@
zend_hash_init(statement-binds, 13, NULL, 
_mssql_bind_hash_dtor, 0);
}
 
-   memset((void*)bind,0,sizeof(mssql_bind));
-   
zend_hash_add(statement-binds,Z_STRVAL_PP(param_name),Z_STRLEN_PP(param_name),bind,sizeof(mssql_bind),(void
 **)bindp);
-   if( NULL == bindp ) RETURN_FALSE;
-   bindp-zval=*var;
-   zval_add_ref(var);
-
-   /* no call to dbrpcparam if RETVAL */
-   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

-   if (dbrpcparam(mssql_ptr-link, Z_STRVAL_PP(param_name), 
(BYTE)status, type, maxlen, datalen, (LPBYTE)value)==FAIL) {
-   php_error_docref(NULL TSRMLS_CC, E_WARNING, Unable to 
set parameter);
-   RETURN_FALSE;
+   if (zend_hash_exists(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name))) {
+   RETURN_FALSE;
+   }
+   else {
+   memset((void*)bind,0,sizeof(mssql_bind));
+   zend_hash_add(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name), bind, sizeof(mssql_bind), (void **)bindp);
+   if( NULL == bindp ) RETURN_FALSE;
+   bindp-zval=*var;
+   zval_add_ref(var);
+   
+   /* no call to dbrpcparam if RETVAL */
+   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

+   if (dbrpcparam(mssql_ptr-link, 
Z_STRVAL_PP(param_name), (BYTE)status, type, maxlen, datalen, 
(LPBYTE)value)==FAIL) {
+   php_error_docref(NULL TSRMLS_CC, E_WARNING, 
Unable to set parameter);
+   RETURN_FALSE;
+   }
}
}
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:41:03 2005 EDT

  Modified files:  (Branch: PHP_5_0)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #32009 crash when mssql_bind() is called more than once
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.517r2=1.1760.2.518ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.517 php-src/NEWS:1.1760.2.518
--- php-src/NEWS:1.1760.2.517   Fri Nov 18 14:43:03 2005
+++ php-src/NEWSFri Nov 18 15:41:02 2005
@@ -59,6 +59,7 @@
 - Fixed bug #32179 (xmlrpc_encode() segfaults with recursive references). 
(Tony)
 - Fixed bug #29983 (PHP does not explicitly set mime type  charset). (Ilia)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
+- Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 
 05 Sep 2005, PHP 5.0.5
 - Upgraded PCRE library to version 5.0. (Andrei)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.137.2.12r2=1.137.2.13ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.137.2.12 
php-src/ext/mssql/php_mssql.c:1.137.2.13
--- php-src/ext/mssql/php_mssql.c:1.137.2.12Fri Nov 18 14:43:15 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:41:03 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.137.2.12 2005/11/18 19:43:15 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.137.2.13 2005/11/18 20:41:03 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2083,17 +2083,22 @@
zend_hash_init(statement-binds, 13, NULL, 
_mssql_bind_hash_dtor, 0);
}
 
-   memset((void*)bind,0,sizeof(mssql_bind));
-   
zend_hash_add(statement-binds,Z_STRVAL_PP(param_name),Z_STRLEN_PP(param_name),bind,sizeof(mssql_bind),(void
 **)bindp);
-   if( NULL == bindp ) RETURN_FALSE;
-   bindp-zval=*var;
-   zval_add_ref(var);
-
-   /* no call to dbrpcparam if RETVAL */
-   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

-   if (dbrpcparam(mssql_ptr-link, Z_STRVAL_PP(param_name), 
(BYTE)status, type, maxlen, datalen, (LPBYTE)value)==FAIL) {
-   php_error_docref(NULL TSRMLS_CC, E_WARNING, Unable to 
set parameter);
-   RETURN_FALSE;
+   if (zend_hash_exists(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name))) {
+   RETURN_FALSE;
+   }
+   else {
+   memset((void*)bind,0,sizeof(mssql_bind));
+   zend_hash_add(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name), bind, sizeof(mssql_bind), (void **)bindp);
+   if( NULL == bindp ) RETURN_FALSE;
+   bindp-zval=*var;
+   zval_add_ref(var);
+   
+   /* no call to dbrpcparam if RETVAL */
+   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

+   if (dbrpcparam(mssql_ptr-link, 
Z_STRVAL_PP(param_name), (BYTE)status, type, maxlen, datalen, 
(LPBYTE)value)==FAIL) {
+   php_error_docref(NULL TSRMLS_CC, E_WARNING, 
Unable to set parameter);
+   RETURN_FALSE;
+   }
}
}
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:41:52 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #32009 crash when mssql_bind() is called more than once
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.85r2=1.1247.2.920.2.86ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.85 php-src/NEWS:1.1247.2.920.2.86
--- php-src/NEWS:1.1247.2.920.2.85  Fri Nov 18 14:15:11 2005
+++ php-src/NEWSFri Nov 18 15:41:49 2005
@@ -3,6 +3,7 @@
 23 Nov 2005, Version 4.4.2
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
+- Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 
 18 Nov 2005, Version 4.4.2RC1
 - Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.86.2.44.2.5r2=1.86.2.44.2.6ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.86.2.44.2.5 
php-src/ext/mssql/php_mssql.c:1.86.2.44.2.6
--- php-src/ext/mssql/php_mssql.c:1.86.2.44.2.5 Fri Nov 18 14:15:12 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:41:51 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.86.2.44.2.5 2005/11/18 19:15:12 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.86.2.44.2.6 2005/11/18 20:41:51 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2130,17 +2130,22 @@
zend_hash_init(statement-binds, 13, NULL, 
_mssql_bind_hash_dtor, 0);
}
 
-   memset((void*)bind,0,sizeof(mssql_bind));
-   
zend_hash_add(statement-binds,Z_STRVAL_PP(param_name),Z_STRLEN_PP(param_name),bind,sizeof(mssql_bind),(void
 **)bindp);
-   if( NULL == bindp ) RETURN_FALSE;
-   bindp-zval=*var;
-   zval_add_ref(var);
-
-   /* no call to dbrpcparam if RETVAL */
-   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

-   if (dbrpcparam(mssql_ptr-link, Z_STRVAL_PP(param_name), 
(BYTE)status, type, maxlen, datalen, (LPBYTE)value)==FAIL) {
-   php_error_docref(NULL TSRMLS_CC, E_WARNING, Unable to 
set parameter);
-   RETURN_FALSE;
+   if (zend_hash_exists(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name))) {
+   RETURN_FALSE;
+   }
+   else {
+   memset((void*)bind,0,sizeof(mssql_bind));
+   zend_hash_add(statement-binds, Z_STRVAL_PP(param_name), 
Z_STRLEN_PP(param_name), bind, sizeof(mssql_bind), (void **)bindp);
+   if( NULL == bindp ) RETURN_FALSE;
+   bindp-zval=*var;
+   zval_add_ref(var);
+   
+   /* no call to dbrpcparam if RETVAL */
+   if ( strcmp(RETVAL,Z_STRVAL_PP(param_name))!=0 ) {

+   if (dbrpcparam(mssql_ptr-link, 
Z_STRVAL_PP(param_name), (BYTE)status, type, maxlen, datalen, 
(LPBYTE)value)==FAIL) {
+   php_error_docref(NULL TSRMLS_CC, E_WARNING, 
Unable to set parameter);
+   RETURN_FALSE;
+   }
}
}
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:48:23 2005 EDT

  Modified files:  
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  Fix #33963. mssql_bind fails on input parameters
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2081r2=1.2082ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2081 php-src/NEWS:1.2082
--- php-src/NEWS:1.2081 Fri Nov 18 15:40:19 2005
+++ php-src/NEWSFri Nov 18 15:48:19 2005
@@ -28,3 +28,4 @@
 - Fixed bug #34286 (__toString() behavior is inconsistent). (Marcus)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.157r2=1.158ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.157 php-src/ext/mssql/php_mssql.c:1.158
--- php-src/ext/mssql/php_mssql.c:1.157 Fri Nov 18 15:40:28 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:48:22 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.157 2005/11/18 20:40:28 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.158 2005/11/18 20:48:22 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2041,7 +2041,9 @@
type=Z_LVAL_PP(yytype);
is_output=Z_LVAL_PP(yyis_output);
is_null=Z_LVAL_PP(yyis_null);
-   maxlen=Z_LVAL_PP(yymaxlen); 

+   if (is_output) {
+   maxlen=Z_LVAL_PP(yymaxlen);
+   }
}
break;  


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:48:47 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcNEWS 
  Log:
  MFH Fix #33963. mssql_bind fails on input parameters
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.205r2=1.2027.2.206ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.205 php-src/NEWS:1.2027.2.206
--- php-src/NEWS:1.2027.2.205   Fri Nov 18 15:40:51 2005
+++ php-src/NEWSFri Nov 18 15:48:46 2005
@@ -5,6 +5,7 @@
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6
 - Changed function parameter parsing to handle integers in a non-strict fashion

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:49:02 2005 EDT

  Modified files:  (Branch: PHP_5_0)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH Fix #33963. mssql_bind fails on input parameters
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.518r2=1.1760.2.519ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.518 php-src/NEWS:1.1760.2.519
--- php-src/NEWS:1.1760.2.518   Fri Nov 18 15:41:02 2005
+++ php-src/NEWSFri Nov 18 15:49:00 2005
@@ -60,6 +60,7 @@
 - Fixed bug #29983 (PHP does not explicitly set mime type  charset). (Ilia)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
 
 05 Sep 2005, PHP 5.0.5
 - Upgraded PCRE library to version 5.0. (Andrei)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.137.2.13r2=1.137.2.14ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.137.2.13 
php-src/ext/mssql/php_mssql.c:1.137.2.14
--- php-src/ext/mssql/php_mssql.c:1.137.2.13Fri Nov 18 15:41:03 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:49:01 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.137.2.13 2005/11/18 20:41:03 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.137.2.14 2005/11/18 20:49:01 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2010,7 +2010,9 @@
type=Z_LVAL_PP(yytype);
is_output=Z_LVAL_PP(yyis_output);
is_null=Z_LVAL_PP(yyis_null);
-   maxlen=Z_LVAL_PP(yymaxlen); 

+   if (is_output) {
+   maxlen=Z_LVAL_PP(yymaxlen);
+   }
}
break;  


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 15:49:18 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH Fix #33963. mssql_bind fails on input parameters
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.86r2=1.1247.2.920.2.87ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.86 php-src/NEWS:1.1247.2.920.2.87
--- php-src/NEWS:1.1247.2.920.2.86  Fri Nov 18 15:41:49 2005
+++ php-src/NEWSFri Nov 18 15:49:15 2005
@@ -4,6 +4,7 @@
 - Fixed bug #35278 (Multiple virtual() calls crash Apache 2 php module). (Ilia)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
 
 18 Nov 2005, Version 4.4.2RC1
 - Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.86.2.44.2.6r2=1.86.2.44.2.7ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.86.2.44.2.6 
php-src/ext/mssql/php_mssql.c:1.86.2.44.2.7
--- php-src/ext/mssql/php_mssql.c:1.86.2.44.2.6 Fri Nov 18 15:41:51 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 15:49:16 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.86.2.44.2.6 2005/11/18 20:41:51 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.86.2.44.2.7 2005/11/18 20:49:16 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -2057,7 +2057,9 @@
type=Z_LVAL_PP(yytype);
is_output=Z_LVAL_PP(yyis_output);
is_null=Z_LVAL_PP(yyis_null);
-   maxlen=Z_LVAL_PP(yymaxlen); 

+   if (is_output) {
+   maxlen=Z_LVAL_PP(yymaxlen);
+   }
}
break;  


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src(PHP_5_1) / NEWS

2005-11-18 Thread Antony Dovgal

Frank, you committed only NEWS file in 5.1 branch.

On 18.11.2005 23:48, Frank M. Kromann wrote:

fmk Fri Nov 18 15:48:47 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-src	NEWS 
  Log:

  MFH Fix #33963. mssql_bind fails on input parameters
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.205r2=1.2027.2.206ty=u

Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.205 php-src/NEWS:1.2027.2.206
--- php-src/NEWS:1.2027.2.205   Fri Nov 18 15:40:51 2005
+++ php-src/NEWSFri Nov 18 15:48:46 2005
@@ -5,6 +5,7 @@
 - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6

 - Changed function parameter parsing to handle integers in a non-strict fashion




--
Wbr, 
Antony Dovgal


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 16:22:51 2005 EDT

  Modified files:  
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  Fix #33201 Crash when fetching some data types
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2082r2=1.2083ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2082 php-src/NEWS:1.2083
--- php-src/NEWS:1.2082 Fri Nov 18 15:48:19 2005
+++ php-src/NEWSFri Nov 18 16:22:40 2005
@@ -29,3 +29,4 @@
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 - Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
+- Fixed bug #33201 (Crash when fetching some data types). (Frank)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.158r2=1.159ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.158 php-src/ext/mssql/php_mssql.c:1.159
--- php-src/ext/mssql/php_mssql.c:1.158 Fri Nov 18 15:48:22 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 16:22:50 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.158 2005/11/18 20:48:22 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.159 2005/11/18 21:22:50 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -891,11 +891,9 @@
if ((column_type != SQLDATETIME  column_type 
!= SQLDATETIM4) || MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -903,6 +901,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}
 
@@ -961,11 +961,9 @@
if ((column_type != SQLDATETIME  column_type != SQLDATETIM4) 
|| MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -973,6 +971,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_1) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 16:23:21 2005 EDT

  Modified files:  (Branch: PHP_5_1)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33201 Crash when fetching some data types
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.206r2=1.2027.2.207ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.206 php-src/NEWS:1.2027.2.207
--- php-src/NEWS:1.2027.2.206   Fri Nov 18 15:48:46 2005
+++ php-src/NEWSFri Nov 18 16:23:19 2005
@@ -6,6 +6,7 @@
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 - Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
+- Fixed bug #33201 (Crash when fetching some data types). (Frank)
 
 17 Nov 2005, PHP 5.1 Release Candidate 6
 - Changed function parameter parsing to handle integers in a non-strict fashion
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.152.2.6r2=1.152.2.7ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.152.2.6 
php-src/ext/mssql/php_mssql.c:1.152.2.7
--- php-src/ext/mssql/php_mssql.c:1.152.2.6 Fri Nov 18 15:40:52 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 16:23:20 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.152.2.6 2005/11/18 20:40:52 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.152.2.7 2005/11/18 21:23:20 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -891,11 +891,9 @@
if ((column_type != SQLDATETIME  column_type 
!= SQLDATETIM4) || MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -903,6 +901,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}
 
@@ -961,11 +961,9 @@
if ((column_type != SQLDATETIME  column_type != SQLDATETIM4) 
|| MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -973,6 +971,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_5_0) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 16:23:46 2005 EDT

  Modified files:  (Branch: PHP_5_0)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33201 Crash when fetching some data types
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1760.2.519r2=1.1760.2.520ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1760.2.519 php-src/NEWS:1.1760.2.520
--- php-src/NEWS:1.1760.2.519   Fri Nov 18 15:49:00 2005
+++ php-src/NEWSFri Nov 18 16:23:45 2005
@@ -61,6 +61,7 @@
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 - Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
+- Fixed bug #33201 (Crash when fetching some data types). (Frank)
 
 05 Sep 2005, PHP 5.0.5
 - Upgraded PCRE library to version 5.0. (Andrei)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.137.2.14r2=1.137.2.15ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.137.2.14 
php-src/ext/mssql/php_mssql.c:1.137.2.15
--- php-src/ext/mssql/php_mssql.c:1.137.2.14Fri Nov 18 15:49:01 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 16:23:45 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.137.2.14 2005/11/18 20:49:01 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.137.2.15 2005/11/18 21:23:45 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -865,11 +865,9 @@
if ((column_type != SQLDATETIME  column_type 
!= SQLDATETIM4) || MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -877,6 +875,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}
 
@@ -935,11 +935,9 @@
if ((column_type != SQLDATETIME  column_type != SQLDATETIM4) 
|| MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -947,6 +945,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) / NEWS /ext/mssql php_mssql.c

2005-11-18 Thread Frank M. Kromann
fmk Fri Nov 18 16:24:07 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-srcNEWS 
/php-src/ext/mssql  php_mssql.c 
  Log:
  MFH: Fix #33201 Crash when fetching some data types
  
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.1247.2.920.2.87r2=1.1247.2.920.2.88ty=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.1247.2.920.2.87 php-src/NEWS:1.1247.2.920.2.88
--- php-src/NEWS:1.1247.2.920.2.87  Fri Nov 18 15:49:15 2005
+++ php-src/NEWSFri Nov 18 16:24:05 2005
@@ -5,6 +5,7 @@
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
 - Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
+- Fixed bug #33201 (Crash when fetching some data types). (Frank)
 
 18 Nov 2005, Version 4.4.2RC1
 - Added missing safe_mode/open_basedir checks into cURL extension. (Ilia)
http://cvs.php.net/diff.php/php-src/ext/mssql/php_mssql.c?r1=1.86.2.44.2.7r2=1.86.2.44.2.8ty=u
Index: php-src/ext/mssql/php_mssql.c
diff -u php-src/ext/mssql/php_mssql.c:1.86.2.44.2.7 
php-src/ext/mssql/php_mssql.c:1.86.2.44.2.8
--- php-src/ext/mssql/php_mssql.c:1.86.2.44.2.7 Fri Nov 18 15:49:16 2005
+++ php-src/ext/mssql/php_mssql.c   Fri Nov 18 16:24:06 2005
@@ -16,7 +16,7 @@
+--+
  */
 
-/* $Id: php_mssql.c,v 1.86.2.44.2.7 2005/11/18 20:49:16 fmk Exp $ */
+/* $Id: php_mssql.c,v 1.86.2.44.2.8 2005/11/18 21:24:06 fmk Exp $ */
 
 #ifdef COMPILE_DL_MSSQL
 #define HAVE_MSSQL 1
@@ -891,11 +891,9 @@
if ((column_type != SQLDATETIME  column_type 
!= SQLDATETIM4) || MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -903,6 +901,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}

@@ -965,11 +965,9 @@
if ((column_type != SQLDATETIME  column_type != SQLDATETIM4) 
|| MS_SQL_G(datetimeconvert)) {
 
switch (column_type) {
-   case SQLDATETIM4 :
-   res_length += 14;
-   break;
case SQLDATETIME :
-   res_length += 10;
+   case SQLDATETIM4 :
+   res_length += 20;
break;
case SQLMONEY :
case SQLMONEY4 :
@@ -977,6 +975,8 @@
case SQLDECIMAL :
case SQLNUMERIC :
res_length += 5;
+   case 127 :
+   res_length += 20;
break;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src(PHP_5_1) / NEWS

2005-11-18 Thread Frank M. Kromann
The fix was already in the code. It was included in my previous commit.

- Frank

 Frank, you committed only NEWS file in 5.1 branch.
 
 On 18.11.2005 23:48, Frank M. Kromann wrote:
  fmk Fri Nov 18 15:48:47 2005 EDT
  
Modified files:  (Branch: PHP_5_1)
  /php-srcNEWS 
Log:
MFH Fix #33963. mssql_bind fails on input parameters

 
http://cvs.php.net/diff.php/php-src/NEWS?r1=1.2027.2.205r2=1.2027.2.206ty=u
  Index: php-src/NEWS
  diff -u php-src/NEWS:1.2027.2.205 php-src/NEWS:1.2027.2.206
  --- php-src/NEWS:1.2027.2.205   Fri Nov 18 15:40:51 2005
  +++ php-src/NEWSFri Nov 18 15:48:46 2005
  @@ -5,6 +5,7 @@
   - Fixed bug #35273 (Error in mapping soap - java types). (Dmitry)
   - Fixed bug #33153 (crash in mssql_next result). (Frank)
   - Fixed bug #32009 (crash when mssql_bind() is called more than
once). (Frank)
  +- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)
   
   17 Nov 2005, PHP 5.1 Release Candidate 6
   - Changed function parameter parsing to handle integers in a
non-strict fashion
  
 
 
 -- 
 Wbr, 
 Antony Dovgal
 
 -- 
 PHP CVS Mailing List (http://www.php.net/)
 To unsubscribe, visit: http://www.php.net/unsub.php
 
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Marcus Boerger
Hello Derick,


 the other config options are not that dangerous. I'd say Jani should add
that info. And maybe we should require =yes or =version so that ppl have
to think about this switch before it works

marcus

Friday, November 18, 2005, 4:40:34 PM, you wrote:

 On Fri, 18 Nov 2005, Antony Dovgal wrote:

 We need at least document it or to add something like this:

 But then we need to do this for every configure option... as nobody 
 really knows what they are doing anyway.

 Derick

 -- 
 Derick Rethans
 http://derickrethans.nl | http://ez.no | http://xdebug.org




Best regards,
 Marcus

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Marcus Boerger
Hello Derick,

  any good ideas on the howto front?

regards
marcus

Friday, November 18, 2005, 9:08:38 AM, you wrote:

 On Thu, 17 Nov 2005, Marcus Boerger wrote:

 helly Thu Nov 17 18:21:15 2005 EDT
 
   Modified files:  
 /php-src/ext/reflection   config.m4 
   Log:
   - Need to define HAVE_REFLECTION

 I think it should not be possible to disable this extension with a 
 simple configure switch...

 Derick




Best regards,
 Marcus

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src /ext/reflection php_reflection.c

2005-11-18 Thread Marcus Boerger
helly   Fri Nov 18 17:59:09 2005 EDT

  Modified files:  
/php-src/ext/reflection php_reflection.c 
  Log:
  - Disabled globals code in a way that does not affect purist or old compilers
  
http://cvs.php.net/diff.php/php-src/ext/reflection/php_reflection.c?r1=1.190r2=1.191ty=u
Index: php-src/ext/reflection/php_reflection.c
diff -u php-src/ext/reflection/php_reflection.c:1.190 
php-src/ext/reflection/php_reflection.c:1.191
--- php-src/ext/reflection/php_reflection.c:1.190   Thu Nov 17 17:59:38 2005
+++ php-src/ext/reflection/php_reflection.c Fri Nov 18 17:59:07 2005
@@ -19,7 +19,7 @@
+--+
 */
 
-/* $Id: php_reflection.c,v 1.190 2005/11/17 22:59:38 helly Exp $ */
+/* $Id: php_reflection.c,v 1.191 2005/11/18 22:59:07 helly Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include config.h
@@ -50,7 +50,9 @@
 PHPAPI zend_class_entry *reflection_property_ptr;
 PHPAPI zend_class_entry *reflection_extension_ptr;
 
+#if MBO_0
 ZEND_BEGIN_MODULE_GLOBALS(reflection)
+   int dummy;
 ZEND_END_MODULE_GLOBALS(reflection)
 
 #ifdef ZTS
@@ -62,11 +64,8 @@
 extern zend_reflection_globals reflectionglobals;
 #endif
 
-#ifdef COMPILE_DL_REFLECTION
-ZEND_GET_MODULE(reflection)
-#endif
-
 ZEND_DECLARE_MODULE_GLOBALS(reflection)
+#endif /* MBO_0 */
 
 /* Method macros */
 
@@ -4184,17 +4183,10 @@
 }
 /* }}} */
 
-static void reflection_init_globals(zend_reflection_globals *globals) /* {{{ */
-{
-   /* Initialize your global struct */
-} /* }}} */
-
 PHP_MINIT_FUNCTION(reflection) /* {{{ */
 {
zend_class_entry _reflection_entry;
 
-   ZEND_INIT_MODULE_GLOBALS(reflection, reflection_init_globals, NULL);
-
zend_std_obj_handlers = zend_get_std_object_handlers();
memcpy(reflection_object_handlers, zend_get_std_object_handlers(), 
sizeof(zend_object_handlers));
reflection_object_handlers.clone_obj = NULL;
@@ -4278,7 +4270,7 @@
php_info_print_table_start();
php_info_print_table_header(2, Reflection, enabled);
 
-   php_info_print_table_row(2, Version, $Id: php_reflection.c,v 1.190 
2005/11/17 22:59:38 helly Exp $);
+   php_info_print_table_row(2, Version, $Id: php_reflection.c,v 1.191 
2005/11/18 22:59:07 helly Exp $);
 
php_info_print_table_end();
 } /* }}} */

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-CVS] cvs: php-src /ext/reflection config.m4

2005-11-18 Thread Andi Gutmans
As I mentioned to Marcus, it seems odd to me to move reflection in 
the first place. It seems to me to be very much a language feature 
and in my opinion, should always be enabled. I can't really see a 
good reason for this move.


Andi

At 07:40 AM 11/18/2005, Derick Rethans wrote:

On Fri, 18 Nov 2005, Antony Dovgal wrote:

 We need at least document it or to add something like this:

But then we need to do this for every configure option... as nobody
really knows what they are doing anyway.

Derick

--
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src /ext/mbstring mbstring.c

2005-11-18 Thread Rui Hirokawa
hirokawaSat Nov 19 01:32:25 2005 EDT

  Modified files:  
/php-src/ext/mbstring   mbstring.c 
  Log:
  fixed #35253 length of the encoded string vilolates a RFC.
  
http://cvs.php.net/diff.php/php-src/ext/mbstring/mbstring.c?r1=1.226r2=1.227ty=u
Index: php-src/ext/mbstring/mbstring.c
diff -u php-src/ext/mbstring/mbstring.c:1.226 
php-src/ext/mbstring/mbstring.c:1.227
--- php-src/ext/mbstring/mbstring.c:1.226   Wed Sep 21 09:23:12 2005
+++ php-src/ext/mbstring/mbstring.c Sat Nov 19 01:32:20 2005
@@ -17,7 +17,7 @@
+--+
  */
 
-/* $Id: mbstring.c,v 1.226 2005/09/21 13:23:12 iliaa Exp $ */
+/* $Id: mbstring.c,v 1.227 2005/11/19 06:32:20 hirokawa Exp $ */
 
 /*
  * PHP 4 Multibyte String module mbstring
@@ -2294,7 +2294,7 @@
 }
 /* }}} */
 
-/* {{{ proto string mb_encode_mimeheader(string str [, string charset [, 
string transfer-encoding [, string linefeed]]])
+/* {{{ proto string mb_encode_mimeheader(string str [, string charset [, 
string transfer-encoding [, string linefeed [, int indent)
Converts the string to MIME encoded-word in the format of 
=?charset?(B|Q)?encoded_string?= */
 PHP_FUNCTION(mb_encode_mimeheader)
 {
@@ -2306,12 +2306,13 @@
int trans_enc_name_len;
char *linefeed = \r\n;
int linefeed_len;
+   int indent = 0;
 
mbfl_string_init(string);
string.no_language = MBSTRG(current_language);
string.no_encoding = MBSTRG(current_internal_encoding);
 
-   if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, s|sss, (char 
**)string.val, string.len, charset_name, charset_name_len, trans_enc_name, 
trans_enc_name_len, linefeed, linefeed_len) == FAILURE) {
+   if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, s|sssl, (char 
**)string.val, string.len, charset_name, charset_name_len, trans_enc_name, 
trans_enc_name_len, linefeed, linefeed_len, indent) == FAILURE) {
return;
}
 
@@ -2341,7 +2342,7 @@
}
 
mbfl_string_init(result);
-   ret = mbfl_mime_header_encode(string, result, charset, transenc, 
linefeed, 0);
+   ret = mbfl_mime_header_encode(string, result, charset, transenc, 
linefeed, indent);
if (ret != NULL) {
RETVAL_STRINGL((char *)ret-val, ret-len, 0)   /* the string 
is already strdup()'ed */
} else {

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



[PHP-CVS] cvs: php-src(PHP_4_4) /ext/mbstring mbstring.c

2005-11-18 Thread Rui Hirokawa
hirokawaSat Nov 19 01:39:43 2005 EDT

  Modified files:  (Branch: PHP_4_4)
/php-src/ext/mbstring   mbstring.c 
  Log:
  MFH: fixed #35253 length of the encoded string vilolates a RFC.
  
http://cvs.php.net/diff.php/php-src/ext/mbstring/mbstring.c?r1=1.142.2.47.2.3r2=1.142.2.47.2.4ty=u
Index: php-src/ext/mbstring/mbstring.c
diff -u php-src/ext/mbstring/mbstring.c:1.142.2.47.2.3 
php-src/ext/mbstring/mbstring.c:1.142.2.47.2.4
--- php-src/ext/mbstring/mbstring.c:1.142.2.47.2.3  Fri Nov  4 20:14:05 2005
+++ php-src/ext/mbstring/mbstring.c Sat Nov 19 01:39:39 2005
@@ -17,7 +17,7 @@
+--+
  */
 
-/* $Id: mbstring.c,v 1.142.2.47.2.3 2005/11/05 01:14:05 hirokawa Exp $ */
+/* $Id: mbstring.c,v 1.142.2.47.2.4 2005/11/19 06:39:39 hirokawa Exp $ */
 
 /*
  * PHP4 Multibyte String module mbstring
@@ -2924,16 +2924,17 @@
 }
 /* }}} */
 
-/* {{{ proto string mb_encode_mimeheader(string str [, string charset [, 
string transfer-encoding [, string linefeed]]])
+/* {{{ proto string mb_encode_mimeheader(string str [, string charset [, 
string transfer-encoding [, string linefeed [, int indent)
Converts the string to MIME encoded-word in the format of 
=?charset?(B|Q)?encoded_string?= */
 PHP_FUNCTION(mb_encode_mimeheader)
 {
-   pval **argv[4];
+   pval **argv[5];
enum mbfl_no_encoding charset, transenc;
mbfl_string  string, result, *ret;
char *p, *linefeed;
+   int indent;
 
-   if (ZEND_NUM_ARGS()  1 || ZEND_NUM_ARGS()  4 || 
zend_get_parameters_array_ex(ZEND_NUM_ARGS(), argv) == FAILURE) {
+   if (ZEND_NUM_ARGS()  1 || ZEND_NUM_ARGS()  5 || 
zend_get_parameters_array_ex(ZEND_NUM_ARGS(), argv) == FAILURE) {
WRONG_PARAM_COUNT;
}
 
@@ -2970,6 +2971,12 @@
linefeed = Z_STRVAL_PP(argv[3]);
}
 
+   indent = 0;
+   if (ZEND_NUM_ARGS() = 5) {
+   convert_to_long_ex(argv[4]);
+   indent = Z_LVAL_PP(argv[4]);
+   }
+
convert_to_string_ex(argv[0]);
mbfl_string_init(string);
mbfl_string_init(result);
@@ -2977,7 +2984,7 @@
string.no_encoding = MBSTRG(current_internal_encoding);
string.val = Z_STRVAL_PP(argv[0]);
string.len = Z_STRLEN_PP(argv[0]);
-   ret = mbfl_mime_header_encode(string, result, charset, transenc, 
linefeed, 0);
+   ret = mbfl_mime_header_encode(string, result, charset, transenc, 
linefeed, indent);
if (ret != NULL) {
RETVAL_STRINGL((char *)ret-val, ret-len, 0)   /* the string 
is already strdup()'ed */
} else {

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php