svn commit: r365828 - projects/nfs-over-tls/usr.sbin/rpc.tlsservd

Rick Macklem rmacklem at FreeBSD.org
Thu Sep 17 02:03:52 UTC 2020


Author: rmacklem
Date: Thu Sep 17 02:03:51 2020
New Revision: 365828
URL: https://svnweb.freebsd.org/changeset/base/365828

Log:
  Oops, used the wrong free() function to free a X509 certificate.

Modified:
  projects/nfs-over-tls/usr.sbin/rpc.tlsservd/rpc.tlsservd.c

Modified: projects/nfs-over-tls/usr.sbin/rpc.tlsservd/rpc.tlsservd.c
==============================================================================
--- projects/nfs-over-tls/usr.sbin/rpc.tlsservd/rpc.tlsservd.c	Thu Sep 17 01:23:16 2020	(r365827)
+++ projects/nfs-over-tls/usr.sbin/rpc.tlsservd/rpc.tlsservd.c	Thu Sep 17 02:03:51 2020	(r365828)
@@ -587,6 +587,8 @@ rpctlssd_verbose_out("get_shutdown1=%d\n", ret);
 				SSL_shutdown(slp->ssl);
 		}
 		SSL_free(slp->ssl);
+		if (slp->cert != NULL)
+			X509_free(slp->cert);
 		/*
 		 * For RPC-over-TLS, this upcall is expected
 		 * to close off the socket.
@@ -594,7 +596,6 @@ rpctlssd_verbose_out("get_shutdown1=%d\n", ret);
 		if (!slp->shutoff)
 			shutdown(slp->s, SHUT_WR);
 		close(slp->s);
-		free(slp->cert);
 		free(slp);
 		result->reterr = RPCTLSERR_OK;
 	} else


More information about the svn-src-projects mailing list