[gnutls-devel] GnuTLS | Fix issues in record_size_limit extension handling (!879)

Development of GNU's TLS library gnutls-devel at lists.gnutls.org
Fri Feb 8 16:53:43 CET 2019


Daiki Ueno commented on a discussion on lib/ext/record_size_limit.c:

>  
> -	/* we do not want to accept sizes less than our minimum */
> -	if (new_size < MIN_RECORD_SIZE)
> -		return 0;
> +	session->internals.hsk_flags |= HSK_RECORD_SIZE_LIMIT_RECEIVED;
>  
> -	/* if a record size larger than user specified maximum, ignore it */
> -	if (new_size > session->security_parameters.max_record_send_size)
> -		return 0;
> +	/* we do not want to accept sizes outside of our supported range */
> +	if (new_size < MIN_RECORD_SIZE) {
> +		/* for server, reject it by omitting the extension in the reply */
> +		if (session->security_parameters.entity == GNUTLS_SERVER)
> +			return gnutls_assert_val(0);
> +		else
> +			return gnutls_assert_val(GNUTLS_E_RECEIVED_ILLEGAL_PARAMETER);

That's a good point; added log calls.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/merge_requests/879#note_139624255
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/20190208/94dfd895/attachment-0001.html>


More information about the Gnutls-devel mailing list