Pierre, Pierre,
+ if (SSL_CTX_use_PrivateKey_file(ctx, resolved_path_buff, SSL_FILETYPE_PEM) != 1) {

this is what the revert gives back, if you go and check what this function does:

"The SSL_CTX_use_PrivateKey_file function loads the private key for use with Secure Sockets Layer (SSL) sessions using a specific context (CTX) structure."

However, what gets passed is path to a certificate, not to a private key. So you reintroduce a bug, that is.

And locally I reverted the patch that was reverting my changes, thus introducing them again, and I got :
Number of tests :   41                38
Tests skipped   :    3 (  7.3%) --------
Tests warned    :    0 (  0.0%) (  0.0%)
Tests failed    :    0 (  0.0%) (  0.0%)
Expected fail   :    0 (  0.0%) (  0.0%)
Tests passed    :   38 ( 92.7%) (100.0%)
---------------------------------------------------------------------
Time taken      :    3 seconds
=====================================================================


So, I am going to revert the revert and reintroduce the code that fixes a bug.

Have a nice day!

Andrey


Pierre Joye wrote:
hi,

On Fri, Apr 23, 2010 at 12:14 PM, Andrey Hristov <p...@hristov.com> wrote:
 More info about the segfaults? Tests that show the segfaults and thus keep
us from regressions?

The tests we have in ext/openssl/tests crash.

However we were wondering why you did these changes and I did not see
any relation between the commit msg and this change. If there is a bug
in this code, please provide a reproduce case and a patch so we can be
sure it won't break ssl functions.

Cheers,


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

Reply via email to