On Tue, 3 Jun 2025, Jeremy Drake via Cygwin wrote:

> On Sun, 1 Jun 2025, Jon Turney wrote:
>
> > On 25/05/2025 17:53, Jeremy Drake via Cygwin wrote:
> > > On Sun, 25 May 2025, 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.
> > >
> > > Lazka pointed out in
> > > https://github.com/msys2/MSYS2-packages/pull/3448#issuecomment-2907899110
> > > that openssl is using pthread tls destructors to do cleanup when
> > > available, so it may not actually be a leak on Cygwin.
> > Wow, that links back to [1]: Now I feel old.
> >
> > [1] https://cygwin.com/pipermail/cygwin/2011-March/194414.html
> >
> > Since I still have the "Building DLLs on Cygwin" part of the manual open in 
> > my
> > editor, I guess maybe I should add "Don't try to use Cygwin's POSIX API 
> > inside
> > the DllMain function" to that...
> >
> > (and more generally, use POSIX mechanisms for initialization/cleanup on
> > process/thread creation/destruction).
> >
>
> I opened a PR against curl, which was merged in
> https://github.com/curl/curl/commit/7860f575fe9e527ced66b31ec076914bbadd64a4

Oops, wrong link, that should be
https://github.com/curl/curl/commit/7a30481760f1f7bfb7928d0b22d49a5ae263386f

-- 
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

Reply via email to