Bug #91774 for Net-SSLeay: Remove SSL_get_tlsa_record

Tue Dec 31 19:05:22 2013 david [...] weekly.org - Ticket created

Subject:

Remove SSL_get_tlsa_record_byname

SSL_get_tlsa_record_byname is not included in OpenSSL git master; it was added optimistically in May to Net::SSLeay per http://comments.gmane.org/gmane.ietf.dane/653 but should probably be removed now to avoid "undefined symbol: SSL_get_tlsa_record_byname at /usr/lib/perl/5.14/DynaLoader.pm line 184." errors when linking against the latest version of OpenSSL (1.1.x DEV). Note that -DOPENSSL_NO_DANE did not result in a valid build. The below patch excises DANE; all tests still pass. --- SSLeay.c~ 2013-12-31 23:54:51.000000000 +0000 +++ SSLeay.c 2014-01-01 00:03:29.000000000 +0000 @@ -14305,31 +14305,7 @@ } #endif -#if OPENSSL_VERSION_NUMBER >= 0x10002000L && !defined(OPENSSL_NO_DANE) -#define XSubPPtmpAACF 1 - -XS(XS_Net__SSLeay_get_tlsa_record_byname); /* prototype to pass -Wmissing-prototypes */ -XS(XS_Net__SSLeay_get_tlsa_record_byname) -{ -#ifdef dVAR - dVAR; dXSARGS; -#else - dXSARGS; -#endif - if (items != 3) - croak_xs_usage(cv, "name, port, type"); - { - char * name = (char *)SvPV_nolen(ST(0)); - int port = (int)SvIV(ST(1)); - int type = (int)SvIV(ST(2)); - - SSL_get_tlsa_record_byname(name, port, type); - } - XSRETURN_EMPTY; -} - -#endif #define REM_EOF "/* EOF - SSLeay.xs */" #ifdef __cplusplus extern "C"

Tue Jan 07 02:29:35 2014 mikem [...] airspayce.com - Correspondence added

Subject:	Re: [rt.cpan.org #91774] Remove SSL_get_tlsa_record_byname
Date:	Tue, 07 Jan 2014 17:29:23 +1000
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	Mike McCauley <mikem [...] airspayce.com>

Hi, thanks for the headsup. Deleted in SVN 391 On Tuesday, December 31, 2013 07:05:23 PM David E. Weekly via RT wrote: Show quoted text

> Tue Dec 31 19:05:22 2013: Request 91774 was acted upon. > Transaction: Ticket created by DEW > Queue: Net-SSLeay > Subject: Remove SSL_get_tlsa_record_byname > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: david@weekly.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=91774 > > > > SSL_get_tlsa_record_byname is not included in OpenSSL git master; it was > added optimistically in May to Net::SSLeay per > http://comments.gmane.org/gmane.ietf.dane/653 but should probably be > removed now to avoid "undefined symbol: SSL_get_tlsa_record_byname at > /usr/lib/perl/5.14/DynaLoader.pm line 184." errors when linking against the > latest version of OpenSSL (1.1.x DEV). Note that -DOPENSSL_NO_DANE did not > result in a valid build. > > The below patch excises DANE; all tests still pass. > > --- SSLeay.c~ 2013-12-31 23:54:51.000000000 +0000 > +++ SSLeay.c 2014-01-01 00:03:29.000000000 +0000 > @@ -14305,31 +14305,7 @@ > } > > #endif > -#if OPENSSL_VERSION_NUMBER >= 0x10002000L && !defined(OPENSSL_NO_DANE) > -#define XSubPPtmpAACF 1 > > - > -XS(XS_Net__SSLeay_get_tlsa_record_byname); /* prototype to pass > -Wmissing-prototypes */ -XS(XS_Net__SSLeay_get_tlsa_record_byname) > -{ > -#ifdef dVAR > - dVAR; dXSARGS; > -#else > - dXSARGS; > -#endif > - if (items != 3) > - croak_xs_usage(cv, "name, port, type"); > - { > - char * name = (char *)SvPV_nolen(ST(0)); > - int port = (int)SvIV(ST(1)); > - int type = (int)SvIV(ST(2)); > - > - SSL_get_tlsa_record_byname(name, port, type); > - } > - XSRETURN_EMPTY; > -} > - > -#endif > #define REM_EOF "/* EOF - SSLeay.xs */" > #ifdef __cplusplus > extern "C"

-- Mike McCauley VK4AMM mikem@airspayce.com Airspayce Pty Ltd 9 Bulbul Place Currumbin Waters QLD 4223 Australia http://www.airspayce.com Phone +61 7 5598-7474 Fax +61 7 5598-7070

Tue Jan 07 02:29:36 2014 The RT System itself - Status changed from 'new' to 'open'

Sat May 24 03:09:32 2014 MIKEM [...] cpan.org - Status changed from 'open' to 'resolved'

Sat May 24 03:09:32 2014 MIKEM [...] cpan.org - Taken

Bug #91774 for Net-SSLeay: Remove SSL_get_tlsa_record_byname