DMARC-Filter: OpenDMARC Filter v1.4.2 delorie.com 54PFCIGs2397653
DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 712FA3858D20
Message-ID: <a83064f3-800e-419c-afca-4791bf8328c0@SystematicSW.ab.ca>
Date: Sun, 25 May 2025 09:11:49 -0600
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: openssl and libcurl issue with cargo
Content-Language: en-CA
To: cygwin@cygwin.com
References: <3a509f81-1b37-8625-7b3a-57b2027ccd66@jdrake.com>
 <083b5ba7-46c8-8780-d362-bea33f227cb9@jdrake.com>
 <874ix9gky6.fsf@Gerda.invalid>
Organization: Systematic Software
In-Reply-To: <874ix9gky6.fsf@Gerda.invalid>
Precedence: list
From: Brian Inglis via Cygwin <cygwin@cygwin.com>
Reply-To: cygwin@cygwin.com
Cc: Brian Inglis <Brian.Inglis@SystematicSW.ab.ca>
Content-Type: text/plain; charset="utf-8"; Format="flowed"
Errors-To: cygwin-bounces~archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces~archive-cygwin=delorie.com@cygwin.com>
Content-Transfer-Encoding: 8bit

On 2025-05-25 02:21, ASSI via Cygwin wrote:
> Jeremy Drake via Cygwin writes:
>> Apparently the fix that MSYS2 made for openssl in
>> https://github.com/msys2/MSYS2-packages/pull/3448  is also not present in
>> Cygwin, so cargo continues to crash after I patched libcurl.
> 
> I have the opposite view on crashing vs. leaking memory from openSSL,
> but the just uploaded test release has the patch.

We are now using totally different versions of OpenSSL with curl &c:

$ curl -V
curl 8.13.0 (x86_64-pc-cygwin) libcurl/8.13.0 OpenSSL/3.0.16 zlib/1.3.1 
brotli/1.1.0 zstd/1.5.7 libidn2/2.3.8 libpsl/0.21.5 libssh2/1.11.0 
nghttp2/1.61.0 libgsasl/2.2.2 OpenLDAP/2.6.9
Release-Date: 2025-04-02
Protocols: dict file ftp ftps gopher gophers http https imap imaps ipfs ipns 
ldap ldaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp ws wss
Features: alt-svc AsynchDNS brotli gsasl GSS-API HSTS HTTP2 HTTPS-proxy IDN IPv6 
Kerberos Largefile libz NTLM PSL SPNEGO SSL threadsafe TLS-SRP UnixSockets zstd

So the issue still exists in Cygwin OpenSSL LTS 3.0.16-1 and ASSI/Stromeko has 
patched it in Cygwin OpenSSL 3.0.16-2 and mingw64-x86_64/i686-openssl 3.0.16-2 
test package releases:

	https://cygwin.com/pipermail/cygwin-announce/2025-May/012341.html

So please try installing the relevant Cygwin/Mingw64 OpenSSL test packages, with 
the latest curl packages, retest and report, if you don't mind.

Perhaps ASSI/Stromeko could push his test release 2 patch to his openssl repo 
playground branch:

	https://cygwin.com/git/?p=git/cygwin-packages/openssl.git

as we get 404 on:

https://cygwin.com/git/?p=git/cygwin-packages/openssl.git;h=refs/heads/playground

so we can confirm that what he actually changed was:

	"remove the ERR_remove_state() call in the DLL_THREAD_DETACH case"

or equivalent, as suggested in:

	https://cygwin.com/pipermail/cygwin/2011-March/194454.html

rather than other suggested hacks elsewhere or with libcurl?

-- 
Take care. Thanks, Brian Inglis              Calgary, Alberta, Canada

La perfection est atteinte                   Perfection is achieved
non pas lorsqu'il n'y a plus rien à ajouter  not when there is no more to add
mais lorsqu'il n'y a plus rien à retrancher  but when there is no more to cut
                                 -- Antoine de Saint-Exupéry

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple