Cvs Commit: cvsnt

[email protected]
Newsgroups gmane.comp.version-control.cvs.windows-nt.cvs
Message-ID <[email protected]>
CVSROOT:	/cvs
Module name:	cvsnt
Changes by:	[email protected]		Thu Jun 10 06:58:23 2010
On host:	debiancvs.march-hare.local

Directory: cvsnt/cvsapi/win32

M SocketIO.cpp	CVSNT_BRANCH_2_8_01_3761	1.1.2.14.4.9.2.10 -> 1.1.2.14.4.9.2.11	Bug Id: 5923

Directory: cvsnt/protocols

M common.cpp	CVSNT_BRANCH_2_8_01_3761	1.37.2.34.6.6.2.4 -> 1.37.2.34.6.6.2.5	Bug Id: 5923

Log message:
Tony writes: I've found another socket problem.. 
there's no error handling so if the other end 
drops the connection the client hangs.   

It's sat on:
> 		WSAEventSelect(tcp_fd,eventback,FD_READ);
> 		waitres = WaitForSingleObject(eventback,
> (tcp_wrwaitx==-1)?INFINITE:(tcp_wrwaitx*1000));
> 



This commit now is trying:
 WSAEventSelect(tcp_fd,eventback,FD_READ|FD_CLOSE);
 waitres = WaitForSingleObject(eventback,(tcp_wrwaitx==-1)?INFINITE:(tcp_wrwaitx*1000));

If I know where the server dropped the 
connection then I can test a crash (div 
zero) there and try it and see...
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.