Ch40zz

Results 14 comments of Ch40zz

https://github.com/curl/curl/pull/13518 sadly did not fix the issue yet, I posted more info on the PR. Needs a bit more investigation, we'll try to provide a repro asap.

Hey, thanks a lot for looking into this! :) We tried this out today, but it sadly did not fix the issue. We're currently not sure yet why or how...

> @Ch40zz can you please try this PR again, I added a [minor change](https://github.com/curl/curl/pull/13518/commits/699f92254ae7bcd49c9cfbe3fbe77b11677c980c) to see if you still get "Invalid Handle" exception Thanks, we'll test it today, will let...

We didn't have much time today so we couldn't get you a repro yet, but we at least ran the new commit and it indeed fixed the double `CloseHandle()` issue....

Quick update from us after testing for a day: This commit is correct and fixes a bug in curl. Our initial bug is still occuring when building with ASAN, however...

> if you can provide some minimal code that exposes the issue, I'll look into it (with asan or not). There is also a possibility that your exit sequence is...

Just a heads up that the bug ticket on MS Developer Community has been created here: https://developercommunity.visualstudio.com/t/ASAN:-heap-use-after-free-in-NdrFullPoin/10654169

Quick update from us after testing for a day: PR https://github.com/curl/curl/pull/13518 is correct and fixes a bug in curl. Our initial bug is still occuring when building with ASAN, however...

> Though I consider this a Windows bug and not a bug in curl I think we should consider reverting GetAddrInfoExW use and go back to the threaded resolver. Thoughts?...

Does the command delay happen on 2.60b too or only on legacy? Did you notice anything you needed to do (e.g. enable a certain feature) to cause the command delay?...