weird loop in s3_lib.c

Michael Richardson <[email protected]> Wed, 03 Jan 2018 18:34:35 -0500
Newsgroups gmane.comp.encryption.openssl.devel
Message-ID <[email protected]>
in
        const SSL_CIPHER *ssl3_choose_cipher(SSL *s, STACK_OF(SSL_CIPHER) *clnt,
                                     STACK_OF(SSL_CIPHER) *srvr)


we have:

    for (i = 0; i < sk_SSL_CIPHER_num(prio); i++) {
        c = sk_SSL_CIPHER_value(prio, i);
        ...

            if (!ok)
                continue;
        }


The if (!ok) continue;
is just a no-op.  Maybe it said at some point:
      "if(ok) break;"

which would have an affect, or maybe there was some code after it that was
removed.

--
]               Never tell me the odds!                 | ipv6 mesh networks [
]   Michael Richardson, Sandelman Software Works        | network architect  [
]     [email protected]  http://www.sandelman.ca/        |   ruby on rails    [

-- 
openssl-dev mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-dev
signature.asc (application/pgp-signature, 487 B)
-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEEbsyLEzg/qUTA43uogItw+93Q3WUFAlpNaIoACgkQgItw+93Q
3WXl8QgAoNICOWSMNrb5wUVK6gv3RoLvhU3i5Glt7aSQx0iDuVVi96CneGrjEvSB
9XwMj2rJKXNOsy3Gc2OR4byFgciTWSSmim6InUeMV7nfRyCqotEV1bim2wkALmHX
+LpYecU0taHvway+pKwyx9wQ3Dtbnp+Q5FGyBCWEdtr5HK245ogMRuqj2nruIxw6
DkgCLm5q8O0vKhXo0XfivLeBmpZgQMsRa23U+Q/fnfrwNWUJkbFzIVrFS+ONYwP8
JxUWeD9edkVwCGEaPXtG+Am4AoakTX8ztQDYkGClDlB6Q6vM/6JAA5MmlBQ31qhP
16HqS6X8Wy8jNSIPt5PqcCgefw7t1Q==
=HdNP
-----END PGP SIGNATURE-----