[gnutls-devel] GnuTLS | Fix removal of duplicate certs during verification (!1653)

Read-only notification of GnuTLS library development activities gnutls-devel at lists.gnutls.org
Tue Oct 25 06:52:39 CEST 2022

Merge request https://gitlab.com/gnutls/gnutls/-/merge_requests/1653 was reviewed by Daiki Ueno

  <!-- Get preloaded note discussion-->
Daiki Ueno started a new discussion on lib/x509/verify-high.c: https://gitlab.com/gnutls/gnutls/-/merge_requests/1653#note_1147254660

> -			ret = cert_set_add(&cert_set, cert_list[i]);
> -			if (ret < 0) {
> +			if (gl_list_nx_add_last(records, cert_list[i]) == NULL) {

nit: a little inconsistency with the above line: `if (gl_list_search(records, cert_list[i + j]))` written without explicit NULL check.

Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/merge_requests/1653
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/20221025/d6eefe93/attachment-0001.html>

More information about the Gnutls-devel mailing list