php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #63767 Follow up for Bug #36103
Submitted: 2012-12-14 08:33 UTC Modified: 2017-10-24 05:01 UTC
Votes:3
Avg. Score:3.0 ± 0.0
Reproduced:2 of 2 (100.0%)
Same Version:0 (0.0%)
Same OS:0 (0.0%)
From: lior dot k at zend dot com Assigned: kaplan (profile)
Status: Assigned Package: FTP related
PHP Version: 5.4.9 OS: Windows
Private report: No CVE-ID: None
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
36 + 25 = ?
Subscribe to this entry?

 
 [2012-12-14 08:33 UTC] lior dot k at zend dot com
Description:
------------
This is a follow up to Bug #36103 ftp ssl connection fails during login

We (at Zend) could still reproduce the problem mentioned in #36103 with PHP 
5.3.19 and 5.4.9 on Windows, and would like to suggest the following patch (PHP 
5.4.9) to resolved the problem.


Patches

windows_ssl_handshake (last revision 2012-12-14 08:33 UTC by lior dot k at zend dot com)

Pull Requests

Pull requests:

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-12-14 08:35 UTC] pajoye@php.net
-Status: Open +Status: Duplicate
 [2012-12-14 08:35 UTC] pajoye@php.net
Please post it at #36103 only, no need to open a new bug. 


Thanks for your work!
 [2012-12-14 08:40 UTC] pajoye@php.net
Ok, reopen, a bit bad to have many bugs for the same issue, but Tony likes it ;-)
 [2012-12-14 08:40 UTC] pajoye@php.net
-Status: Duplicate +Status: Assigned -Assigned To: +Assigned To: tony2001
 [2013-02-10 12:58 UTC] lior dot k at zend dot com
Any news about this ?
 [2017-10-24 05:01 UTC] kalle@php.net
-Assigned To: tony2001 +Assigned To: kaplan
 [2017-10-24 05:01 UTC] kalle@php.net
@lior

As you got karma yourself, you can go ahead and commit the patch of yours, however like noted:

- The code do/while code should be refactored into a function as its re-used
- Please add a test to this fix

Merging this to as low as PHP-7.0 should be perfectly acceptable, thanks!
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Wed Dec 11 00:01:28 2024 UTC