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

[Zebediah Figura]

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.