Re: [PHP-CVS] cvs: php-src(PHP_5_3) /main/streams streams.c

2009-01-08 Thread Antony Dovgal 
On 08.01.2009 21:46, Felipe Pena wrote:
 felipeThu Jan  8 18:46:00 2009 UTC
 
   Modified files:  (Branch: PHP_5_3)
 /php-src/main/streams streams.c 
   Log:
   MFH:
   - Fixed bug #47037 (No error when using fopen with emty string)
   Patch by Cristian Rodriguez R.

Would you mind fixing these tests, too? Thanks.

5_2/ext/tidy/tests/019.phpt
5_2/ext/tidy/tests/022.phpt
5_2/ext/standard/tests/strings/md5_file.phpt
5_2/ext/standard/tests/file/readfile_variation10.phpt
5_2/ext/standard/tests/file/file_put_contents_variation8.phpt
5_2/ext/exif/tests/exif_imagetype_variation1.phpt

-- 
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_3) /main/streams streams.c

2009-01-08 Thread Felipe Pena 
Em Sex, 2009-01-09 às 00:27 +0300, Antony Dovgal escreveu:
 On 08.01.2009 21:46, Felipe Pena wrote:
  felipe  Thu Jan  8 18:46:00 2009 UTC
  
Modified files:  (Branch: PHP_5_3)
  /php-src/main/streams   streams.c 
Log:
MFH:
- Fixed bug #47037 (No error when using fopen with emty string)
Patch by Cristian Rodriguez R.
 
 Would you mind fixing these tests, too? Thanks.
 
 5_2/ext/tidy/tests/019.phpt
 5_2/ext/tidy/tests/022.phpt
 5_2/ext/standard/tests/strings/md5_file.phpt
 5_2/ext/standard/tests/file/readfile_variation10.phpt
 5_2/ext/standard/tests/file/file_put_contents_variation8.phpt
 5_2/ext/exif/tests/exif_imagetype_variation1.phpt
 

Ops, np! I'll fix them. Thanks Tony ;)

-- 
Regards,
Felipe Pena


-- 
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_3) /main/streams streams.c

2008-08-06 Thread Jani Taskinen 

Antony Dovgal kirjoitti:

tony2001Wed Aug  6 09:03:48 2008 UTC

  Modified files:  (Branch: PHP_5_3)
/php-src/main/streams	streams.c 
  Log:

  MFH: increase context refcount before open() to avoid crash when open() fails 
and destroys the context
  
  

[snip]

Isn't this a bug fix which should be MFH'd to PHP_5_2 as well?

--Jani


--
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_3) /main/streams streams.c

2008-08-06 Thread Antony Dovgal 

On 06.08.2008 19:19, Jani Taskinen wrote:

Antony Dovgal kirjoitti:

tony2001Wed Aug  6 09:03:48 2008 UTC

  Modified files:  (Branch: PHP_5_3)
/php-src/main/streams	streams.c 
  Log:

  MFH: increase context refcount before open() to avoid crash when open() fails 
and destroys the context
  
  

[snip]

Isn't this a bug fix which should be MFH'd to PHP_5_2 as well?


No, this is a fix for my previous patch, which I didn't commit to 5_2 
(as it's a bugfix and an improvement in the same time).


--
Wbr, 
Antony Dovgal


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