[Lcdproc] sock_connect() bug

Peter Marschall peter@adpm.de
Sun May 7 11:17:02 2006


Hi Frederick,

On Friday, 5. May 2006 21:43, Frederick Nacino wrote:
> sock_connect() in shared/sockets.c is supposed to return a descriptor
> referencing the socket via the socket() function. 0 is a valid
> descriptor, however the sock_connect() code and a couple of the clients
> contradict that. I've included a patch that correctly assumes any
> descriptor above -1 is valid. People typically shouldn't run into this
> problem unless you finagle with environment as I have. Normally stdin
> will take up descriptor 0 eliminating the case where the socket()
> function can return a 0.

You are right.
Patch committed to CVS (devel & stable)

Thanks for supporting LCDproc
Peter

-- 
Peter Marschall
peter@adpm.de