Re: svn: /php/php-src/ branches/PHP_5_3/ext/openssl/openssl.c trunk/ext/openssl/openssl.c

From: Date: Fri, 23 Apr 2010 11:28:49 +0000
Subject: Re: svn: /php/php-src/ branches/PHP_5_3/ext/openssl/openssl.c trunk/ext/openssl/openssl.c
References: 1 2 3 4  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-62381@lists.php.net to get a copy of this message
On 23.04.2010 15:05, Andrey Hristov wrote:
> "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.

AFAIK the certificate may contain several items, including the private key.
At least that worked fine for me.

> 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
> =====================================================================

Oh, nice!
Try to run ext/openssl/tests/bug46127.phpt with valgrind now.

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

Your fix fixes nothing, please don't reintroduce the segfaults.
If you're unable to reproduce them, I'm ready to do it for you: http://pastebin.com/TPCd7WUU

-- 
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP


Thread (10 messages)

« previous php.cvs (#62381) next »