[gnutls-devel] GnuTLS | set_ciphersuite_list(): Use linear approach to cleanup priorities (!889)

[Development of GNU's TLS library]

Daiki Ueno started a new discussion on lib/priority.c:

>  	 * a pre-TLS1.2 protocol is there; that is because servers which
>  	 * do not support TLS1.3 will negotiate TLS1.2 if seen a TLS1.3 handshake */
>  	if (unlikely((!have_psk && tlsmax && tlsmax->id >= GNUTLS_TLS1_3 && priority_cache->groups.size == 0)) ||
> -	    (!have_tls12 && have_pre_tls12 && have_tls13)) {
> -		for (i = 0; i < priority_cache->protocol.num_priorities; i++) {
> +	    (!have_tls12 && have_pre_tls12 && have_tls13))
> +	{

[Opening brace should be on the previous line](https://www.kernel.org/doc/html/latest/process/coding-style.html#placing-braces-and-spaces).

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/merge_requests/889#note_134237284
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20190123/f8092e21/attachment-0001.html>