[gnutls-devel] GnuTLS | Cleanup and fixes (!779)

Development of GNU's TLS library gnutls-devel at lists.gnutls.org
Fri Oct 19 17:23:02 CEST 2018


Nikos Mavrogiannopoulos started a new discussion on lib/cert-cred.c:

>  void
>  gnutls_certificate_free_credentials(gnutls_certificate_credentials_t sc)
>  {
> -	gnutls_x509_trust_list_deinit(sc->tlist, 1);
> -	gnutls_certificate_free_keys(sc);
> -	memset(sc->pin_tmp, 0, sizeof(sc->pin_tmp));
> +	// Check for valid pointer and otherwise do nothing
> +	if (sc != NULL)	{

This is ok, but adding the check on top such as:
```
if (sc == NULL)
        return;
```
Would reduce the number of changes.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/merge_requests/779#note_110291891
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/20181019/b595cb54/attachment.html>


More information about the Gnutls-devel mailing list