一个FD_SET问题

[greenweather]

有个进程调用这个函数，

int SockCanRead(int iSocketId, int timeout)
{
	fd_set readfds;
	int iMaxFd;
	struct	timeval stuTime;
	int iReady;
	iMaxFd = iSocketId;
	memset((char *)&readfds, 0, sizeof(fd_set));
	FD_ZERO(&readfds);
	FD_SET(iSocketId, &readfds);
	if (timeout > 0){
		stuTime.tv_sec = timeout;
		stuTime.tv_usec = 0;
		iReady = select(iMaxFd + 1, &readfds, NULL, NULL, &stuTime);
	}
	else{
		iReady = select(iMaxFd + 1, &readfds, NULL, NULL, NULL);
	}	
	if (iReady > 0 && FD_ISSET(iSocketId, &readfds)){
		return 0;
	}
	else{
		return -1;
	}
}

执行到FD_SET(iSocketId, &readfds);这句直接导致core了，请教各位大大

[Sou2012]

先把参数的参数检查一下。

[greenweather]

[Quote=引用 6 楼 maoxing63570 的回复:]

你需要给FD_SET的第一个参数传一个套接字，typedef UINT_PTR SOCKET;这个是套接字的类型
[/Quote]
那个套接字有另外一个函数生成的，呵呵，应该不是这个问题

[nfer_cn]

[Quote=引用 8 楼 greenweather 的回复:]

引用 6 楼 maoxing63570 的回复:

你需要给FD_SET的第一个参数传一个套接字，typedef UINT_PTR SOCKET;这个是套接字的类型

那个套接字有另外一个函数生成的，呵呵，应该不是这个问题
[/Quote]
但是要对传入的参数进行判断啊！

[justkk]

这段代码没看出问题

[greenweather]

问题在于如果iSocketId<0，那句才会过不去，进程就core了

[justkk]

哦 你自己已经找到原因了

从头文件中看
#define __FD_SET(d, s) (__FDS_BITS (s)[__FDELT(d)] |= __FDMASK(d))

这样如果d < 0，那么将访问一个无效的下标

[nfer_cn]

memset((char *)&readfds, 0, sizeof(fd_set));
把这行去掉试试

[nfer_cn]

进到函数内部，先判断参数
if (iSocketId == -1)
{
return -1;
}

[maoxing63570]

你需要给FD_SET的第一个参数传一个套接字，typedef UINT_PTR SOCKET;这个是套接字的类型

[greenweather]

如果套接字小于0，然后做write或者read会出问题是吧？