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

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