[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