[PATCH 2/2] kernel32/tests: Close all handles on process exit.

Zebediah Figura z.figura12 at gmail.com
Thu Nov 16 22:20:29 CST 2017


On 11/16/2017 09:56 PM, Dmitry Timoshkov wrote:
>> +    CloseHandle(loader_lock_event);
>> +    CloseHandle(peb_lock_event);
>> +    CloseHandle(heap_lock_event);
>> +    CloseHandle(ack_event);
>> +    CloseHandle(stop_event);
>> +    CloseHandle(process);
>> +    /* Don't close event, mutex and semaphore here; we want to test if they are
>> +     * abandoned on process termination. */
>> +
>>       trace("call ExitProcess(195)\n");
>>       ExitProcess(195);
>>   }
> 
> Please don't add these CloseHandle() calls, they clutter the code and don't do
> anything useful.
> 

Thanks. I've resent the patch just to remove the existing CloseHandle() 
call.



More information about the wine-devel mailing list