Ben Laurie wrote:
> Nils Larsch wrote:
>
>> OpenSSL CVS Repository
>> http://cvs.openssl.org/
>>
>> __________________________________________________ __________________________
>>
>>
>> Server: cvs.openssl.org Name: Nils Larsch
>> Root: /e/openssl/cvs Email: nils@openssl.org
>> Module: openssl Date: 01-Apr-2005 19:49:33
>> Branch: OpenSSL_0_9_7-stable Handle: 2005040118493300
>>
>> Modified files: (Branch: OpenSSL_0_9_7-stable)
>> openssl/ssl ssl_rsa.c
>>
>> Log:
>> really clear the error queue here

>
>
> ? What does this mean? Removing the (void) has no functional effect.


yep, but replacing ERR_get_error() with ERR_clear_error() has :-)

....
>> $ cvs diff -u -r1.16.2.3 -r1.16.2.4 ssl_rsa.c
>> --- openssl/ssl/ssl_rsa.c 11 Aug 2003 18:56:19 -0000 1.16.2.3
>> +++ openssl/ssl/ssl_rsa.c 1 Apr 2005 17:49:33 -0000 1.16.2.4
>> @@ -804,7 +804,7 @@
>> /* When the while loop ends, it's usually just EOF. */
>> err = ERR_peek_last_error();
>> if (ERR_GET_LIB(err) == ERR_LIB_PEM && ERR_GET_REASON(err)
>> == PEM_R_NO_START_LINE)
>> - (void)ERR_get_error();
>> + ERR_clear_error();


Cheers,
Nils
__________________________________________________ ____________________
OpenSSL Project http://www.openssl.org
Development Mailing List openssl-dev@openssl.org
Automated List Manager majordomo@openssl.org