错误的文件描述符：accept（）错误

Question

我正在尝试编写一个简单的聊天应用程序，其中客户端从服务器接收字符串并发送新消息，该消息将传递到下一个连接的客户端。 我是套接字编程的新手，但我真的不知道为什么会发生此错误。 客户端发送缓冲区后，服务器会通知accept()错误（返回-1）和错误的文件描述符。 这是服务器上运行的代码：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>

void error(char *msg) {
    perror(msg);
    exit(1);
}

int main (int argc, char *argv[] ) {

    if ( argc < 2 ) {
        error("ERROR, no port provided\n");
    }

    int sockfd;
    struct sockaddr_in serv_addr, cli_addr;
    int cli_len, portno;
    char message[256];
    pid_t pid;
    int n;

    portno = atoi( argv[1] );
    bzero( (char*)&serv_addr, sizeof(serv_addr) );
    serv_addr.sin_family = AF_INET;
    serv_addr.sin_addr.s_addr = INADDR_ANY;
    serv_addr.sin_port = htons( portno );

    sockfd = socket( AF_INET, SOCK_STREAM, 0 );

    n =  bind ( sockfd, (struct sockaddr*)&serv_addr, sizeof(serv_addr) );
    if ( n < 0 ) {
        error("ERROR on binding\n");
    }

    listen( sockfd, 5 );

    do {
        cli_len = sizeof(cli_addr);

        int newsockfd;
        newsockfd = accept( sockfd,(struct sockaddr*)&cli_addr,(socklen_t *)&cli_len);
        if ( newsockfd < 0 ) {
            printf("%i", newsockfd);
            error("ERROR on accept\n");

        }

        pid = fork();
        if( pid == 0 ) {

            close( sockfd );
            sockfd = -1;

            n = send( newsockfd, message, strlen(message), 0);
            if ( n < 0 ) error("ERROR sending\n");

            bzero(message, 256);

            n = recv( newsockfd, message, 256, 0);
            if ( n < 0 ) error("ERROR receiving\n");

            close( newsockfd );
            newsockfd = -1;

        } 

    }
    while( 1 );

    return 0;
}

Answer 1

如果fork()的结果为零，则需要使用接受的套接字进行I / O， 然后退出process 。 目前，在关闭侦听套接字后，您允许在子进程中重复执行accept循环。

Answer 2

分叉的孩子应该在read()之后结束，但是没有结束。 它继续循环。 可能不是您的意图。

然后，子代调用accept()将-1作为套接字描述符传递。 -1不是有效的套接字描述符。

    accept(sockfd, ...

孩子在这里将sockfd设置为-1 ：

    pid = fork();
    if( pid == 0 ) {

        close( sockfd );
        sockfd = -1;

这样做会丢失socket()返回的有效套接字描述符。

因此，您观察到的错误不是由侦听服务器进程引起的，而是由分叉的孩子引起的。

要解决此问题，请在孩子内部添加对exit()的调用，如下所示：

        ...
        newsockfd = -1;
        exit(EXIT_SUCCESS);
      }

---

此int cli_len; 应该是socklen_t cli_len; 。 并且在调用accept()时应删除cli_len 。 不要盲目抛弃错误！