Date: Thu, 17 Sep 2020 02:03:52 +0000 (UTC) From: Rick Macklem <rmacklem@FreeBSD.org> To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r365828 - projects/nfs-over-tls/usr.sbin/rpc.tlsservd Message-ID: <202009170203.08H23qwD099538@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202009170203.08H23qwD099538>