Hi,
while debugging another thing I discovered what I think is a bug in
the CIPHER list handling.
Can someone please confirm or disprove that this is a bug?

OpenSSL version 1.0.1c.

In function  ssl_cipher_apply_rule the break criteria "(curr == last)
is strange.
I arrive here with only ONE member in my list, so head == tail.
ssl_ciph.c: 956

954:   for (;;)
955:   {
956:     if ((curr == NULL) || (curr == last)) break;


If it breaks on (curr == last),
then a list with only one member will not be processed at all.

I think the check
if (curr == last) break;
should be in the end of the for -loop instead?

best regards
Peter
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
User Support Mailing List                    openssl-users@openssl.org
Automated List Manager                           majord...@openssl.org

Reply via email to