This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch]: secret event


On Fri, Jan 23, 2004 at 10:59:52AM +0100, Corinna Vinschen wrote:
>On Jan 22 18:33, Pierre A. Humblet wrote:
>> 2004-01-22  Pierre Humblet <pierre.humblet@ieee.org>
>> 
>> 	* fhandler_socket.cc (fhandler_socket::create_secret_event): Avoid
>> 	creating multiple handles. Always allow event inheritance but set the
>> 	handle inheritance appropriately. Improve error handling.
>> 	(fhandler_socket::check_peer_secret_event): Improve error handling.
>> 	(fhandler_socket::close_secret_event): Simply call CloseHandle.
>> 	(fhandler_socket::set_close_on_exec): Set secret event inheritance.
>
>Looks good.  Please check it in.

I agree, with one nit.  Was there a reason for getting rid of the handle
protection in this patch?  We are apparently stumbling over a problem with
handle corruption in the current CVS so removing a chance for protection
seems like we're going backwards.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]