Bug #75031 for Net-SSLeay: Broken functions (I do not believe that these ever work correctly)

Wed Feb 15 09:56:46 2012 kmx [...] volny.cz - Ticket created

Subject:	Broken functions (I do not believe that these ever work correctly)
Date:	Wed, 15 Feb 2012 15:56:36 +0100
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

The following functions are IMHO broken and/or badly implemented. These are mostly 1:1 mapping to openssl C API however the functions require some non trivial buffer or callback handling however XS implementation does not handle it. X509_STORE_CTX_get_ex_data X509_STORE_CTX_set_ex_data SESSION_get_app_data SESSION_set_app_data SESSION_get_ex_data SESSION_set_ex_data SESSION_get_ex_new_index CTX_get_app_data CTX_set_app_data CTX_get_ex_data CTX_get_ex_new_index CTX_set_ex_data get_app_data set_app_data get_ex_data set_ex_data get_ex_new_index CTX_callback_ctrl callback_ctrl CTX_ctrl ctrl CTX_set_tmp_dh_callback set_tmp_dh_callback CTX_set_tmp_rsa_callback set_tmp_rsa_callback set_info_callback ** d2i_SSL_SESSION ** i2d_SSL_SESSION ** get_finished ** get_peer_finished ** get_shared_ciphers ** CIPHER_description Items marked ** can be IMHO fixed quite easily (but it will require some changes in calling convention - but I think that these are so badly implemented and undocumented now that nobody is using them). Other items (not marked **) are as far as I can tell simply not working and totally broken (or maybe they are just out of my mental capabilities - in that case somebody should document how to use them). -- kmx

Wed Feb 15 17:03:31 2012 mikem [...] open.com.au - Correspondence added

Subject:	Re: [rt.cpan.org #75031] Broken functions (I do not believe that these ever work correctly)
Date:	Thu, 16 Feb 2012 08:03:24 +1000
To:	bug-Net-SSLeay [...] rt.cpan.org
From:	Mike McCauley <mikem [...] open.com.au>

Hi, That seems OK. I have certainly never used any of these, so I dont know how broken they actually are. Cheers. On Wednesday, February 15, 2012 09:56:46 AM you wrote: Show quoted text

> Wed Feb 15 09:56:46 2012: Request 75031 was acted upon. > Transaction: Ticket created by kmx@volny.cz > Queue: Net-SSLeay > Subject: Broken functions (I do not believe that these ever work > correctly) Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: kmx@volny.cz > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=75031 > > > > The following functions are IMHO broken and/or badly implemented. These > are mostly 1:1 mapping to openssl C API however the functions require > some non trivial buffer or callback handling however XS implementation > does not handle it. > > X509_STORE_CTX_get_ex_data > X509_STORE_CTX_set_ex_data > SESSION_get_app_data > SESSION_set_app_data > SESSION_get_ex_data > SESSION_set_ex_data > SESSION_get_ex_new_index > CTX_get_app_data > CTX_set_app_data > CTX_get_ex_data > CTX_get_ex_new_index > CTX_set_ex_data > get_app_data > set_app_data > get_ex_data > set_ex_data > get_ex_new_index > CTX_callback_ctrl > callback_ctrl > CTX_ctrl > ctrl > CTX_set_tmp_dh_callback > set_tmp_dh_callback > CTX_set_tmp_rsa_callback > set_tmp_rsa_callback > set_info_callback > ** d2i_SSL_SESSION > ** i2d_SSL_SESSION > ** get_finished > ** get_peer_finished > ** get_shared_ciphers > ** CIPHER_description > > Items marked ** can be IMHO fixed quite easily (but it will require some > changes in calling convention - but I think that these are so badly > implemented and undocumented now that nobody is using them). > > Other items (not marked **) are as far as I can tell simply not working > and totally broken (or maybe they are just out of my mental capabilities > - in that case somebody should document how to use them). > > -- > kmx

-- Mike McCauley mikem@open.com.au Open System Consultants Pty. Ltd 9 Bulbul Place Currumbin Waters QLD 4223 Australia http://www.open.com.au Phone +61 7 5598-7474 Fax +61 7 5598-7070 Radiator: the most portable, flexible and configurable RADIUS server anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald, Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS, TTLS, PEAP, TNC, WiMAX, RSA, Vasco, Yubikey, MOTP, HOTP, TOTP, DIAMETER etc. Full source on Unix, Windows, MacOSX, Solaris, VMS, NetWare etc.

Wed Feb 15 17:03:32 2012 The RT System itself - Status changed from 'new' to 'open'

Thu Feb 07 21:11:04 2013 MIKEM [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Feb 07 21:11:05 2013 MIKEM [...] cpan.org - Given to MIKEM