[2/2] wininet: Free per-thread error info upon exit from an async procedure.

Hans Leidekker hans at codeweavers.com
Mon Dec 21 07:42:55 CST 2009


On Monday 21 December 2009 13:09:05 Jacek Caban wrote:

> You have to set TLS value to NULL otherwise it will crash on next 
> INTERNET_SetLastError in the same thread on another async call. But the 

Thanks for catching that.

> right way to fix this is to get rid of INTERNET_[GS]etLastError. This is 
> useless and has proven to result in code that has many failure paths 
> without last error set. I've already removed most of these calls and I'm 
> near to getting rid of all of them except ones in ftp.c.

Well, InternetGetLastResponseInfo depends on the per-thread error info and
it may get called (from a callback) on an async thread, so I'd say we do
need to free the TLS value after the async call is done.

 -Hans



More information about the wine-devel mailing list