3

大家好,我对recv() 有这个奇怪的问题。我正在编写客户端/服务器,客户端发送()一条消息(确切地说是一个结构)和服务器recv()它。我也在使用多个套接字和 select()。

while(1)
{
    readset = info->read_set;
    info->copy_set = info->read_set;

    timeout.tv_sec = 1; 
    timeout.tv_usec = 0; // 0.5 seconds

    ready = select(info->max_fd+1, &readset, NULL, NULL, &timeout);

    if (ready == -1)
    {
        printf("S: ERROR: select(): %s\nEXITING...", strerror(errno));
        exit(1);
    }
    else if (ready == 0)
    {
        continue;
    }
    else
    {
        printf("S: oh finally you have contacted me!\n");
        for(i = 0; i < (info->max_fd+1); i++)
        {

            if(FD_ISSET(i, &readset)) //this is where problem begins
            {
                printf("S: %i is set\n", i);
                printf("S: we talking about socket %i son\n", i);  // i = 4
                num_bytes = recv(i, &msg, MAX_MSG_BYTE, 0);
                printf("S: number of bytes recieved in socket %i is %i\n", i, num_bytes); // prints out i = 0 what??

                if (num_bytes == 0)
                {
                    printf("S: socket has been closed\n");
                    break;
                }
                else if (num_bytes == -1)
                {
                    printf("S: ERROR recv: %d %s \n", i, strerror(errno));
                    continue;
                }
                else                    
                {
                    handle_request(arg, &msg);
                    printf("S: msg says %s\n", msg->_payload);
                }
            } // if (FD_ISSET(i, &readset)
            else
                printf("S:  %i is not set\n", i);
        } // for (i = 0; i < maxfd+1; i++) to check sockets for msg
    } // if (ready == -1)   

    info->read_set = info->copy_set;
    printf("S: copied\n");

} 

我遇到的问题是read_set,0~3 没有设置,4 是。那也行。但是当我打电话时recv()i突然变成0。这是为什么?对我来说,为什么recv()要使用套接字文件描述符编号并修改为另一个编号是没有意义的。这正常吗?我错过了什么吗?

S:  0 is not set
S:  1 is not set
S:  2 is not set
S:  3 is not set
S: 4 is set
S: we talking about socket 4 son
S: i is strangely or unstrangely 0
S: number of bytes recieved in socket 0 is 40

这就是它打印出来的。

4

2 回答 2

2

recv不能修改它的第一个参数,因为它是按值取的。

您没有显示您声明的位置msgor i,而是基于此行

printf("S: msg says %s\n", msg->_payload);

在您使用->运算符 on 的地方msg,我假设它可能是这样的:

struct somestruct* msg = malloc(sizeof(struct somestruct));
int i;

然后你这样做:

num_bytes = recv(i, &msg, MAX_MSG_BYTE, 0);

请注意,msg已经&msg是一个指针,指向指针的指针也是如此。

然后这将做的是接收数据并尝试将其存储在msg 指针本身所在的位置,而不是msg指向位置。通常,指针只有 4 个字节长,因此如果您收到超过 4 个字节,这将溢出存储空间。如果i在 之后在堆栈上声明msg,那么它很可能被此溢出覆盖,并且恰好被接收到的数据包中的所有零字节覆盖。

由于msg已经是一个指针,因此更改您的接收行以消除多余的间接:

num_bytes = recv(i, msg, MAX_MSG_BYTE, 0);

同样,您可能需要考虑对行进行相同的更改

handle_request(arg, &msg)

如果handle_request函数并不真正期望指针指向。

于 2010-04-20T00:29:06.167 回答
1

我的第一个猜测是,sizeof(msg) < MAX_MSG_BYTE当它recv溢出时msg会丢弃i

于 2010-04-20T00:22:23.963 回答