gganis commented on this pull request.
> @@ -1045,6 +1045,7 @@ int XrdCryptosslX509SignProxyReq(XrdCryptoX509 *xcpi, XrdCryptoRSA *kcpi,
// Notify what we added
int crit = X509_EXTENSION_get_critical(xpiextdup);
DEBUG("added extension '"<<s<<"', critical: " << crit);
+ X509_EXTENSION_free( xpiextdup );
Not sure about this. The extension is owned by the proxy certificate (xpo) ...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/862#pullrequestreview-176636710
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
