接受连接后打开两个线程时，C 无限循环中断并出现代码 141

Question

在一个简单的 C 程序中，每次接受 TCP 套接字上的传入连接时，我都会打开一个新线程来异步处理客户端输入。接受是在无限循环中发生的。客户端数据在传递给 pthread_create 的回调函数中进行处理。当客户端发送数据时，套接字将立即关闭并断开连接。

一旦我使用 telnet 客户端连接到程序监听的端口，程序就准备好接受新连接。到目前为止一切顺利。

现在，当我同时连接两个客户端并依次为其提供一些输入时，主程序将退出并显示代码 141。

服务器控制台

cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ ./sysmon 
Thu Apr  9 12:03:17 2015: Finished reading configuration file
Thu Apr  9 12:03:17 2015: Initializing server socket
Thu Apr  9 12:03:17 2015: Accepting connections...
Thu Apr  9 12:03:19 2015: Inbound connection from  127.0.0.1
Using Thread: 0
Thu Apr  9 12:03:19 2015: Accepting connections...
Thu Apr  9 12:03:22 2015: Inbound connection from  127.0.0.1
Using Thread: 1
Thu Apr  9 12:03:22 2015: Accepting connections...
Client msg: asdf
Client msg: asdfasdfdsfadsf
cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ echo $?
141
cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ 

客户端控制台 1

cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ telnet 127.0.0.1 50231
Trying 127.0.0.1...
Connected to 127.0.0.1.
Escape character is '^]'.
asdfasdfdsfadsf
Connection closed by foreign host.
cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ 

客户端控制台 2

cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ telnet 127.0.0.1 50231
Trying 127.0.0.1...
Connected to 127.0.0.1.
Escape character is '^]'.
asdf
Message was: asdf
Connection closed by foreign host.
cehrig@devbox /home/cehrig/projects/SystemMonitor/build $ 

这是用于接受连接的函数的片段。

int connections = 0;
pthread_t * newthread = malloc(sizeof(pthread_t));

while(1) {
    _print(stdout, "messages.socketacceppt", cfg, 1);
    newsockfd = accept(sockfd, (struct sockaddr *) &cli_addr, &clilen);

    thrpass_st thr_pass;
    thr_pass.sockfd = newsockfd;
    thr_pass.cfg = cfg;

    _print(stdout, "messages.socketreceived", cfg, 0);
    fprintf(stdout, "%d.%d.%d.%d\n",
        cli_addr.sin_addr.s_addr&0xFF,
        (cli_addr.sin_addr.s_addr&0xFF00)>>8,
        (cli_addr.sin_addr.s_addr&0xFF0000)>>16,
        (cli_addr.sin_addr.s_addr&0xFF000000)>>24);

    printf("Using Thread: %d\n", connections);
    pthread_create(newthread+connections, NULL, &read_socket, (void *) &thr_pass);

    newthread = (pthread_t *) realloc(newthread, (++connections+1)*sizeof(pthread_t));
}

这是每个新线程的回调函数/入口点。

void * read_socket(void * args)
{
    thrpass_st * _args = (thrpass_st *) malloc(sizeof(thrpass_st));
    _args = (thrpass_st *) args;

    int n;
    char * _buf = (char *) malloc(512*sizeof(char));
    char * _cor = (char *) malloc(512*sizeof(char));
    char * _out = _cor;

    bzero(_buf, 512);
    bzero(_cor, 512);


    size_t bread = 0;
    do {
        if((n = read(_args->sockfd, _buf+bread, 512-bread)) < 0) {
            _print(stderr, "messages.socketreadfail", _args->cfg, 1);
            _exit(0);
        }
        bread+=n;
    } while(strchr(_buf, '\n') == NULL && bread <= 512);


    int x = 0;
    while(*_buf != '\n' && x++ <= 512) {
        *_cor++ = *_buf++;
    }

    printf("Client msg: %s\n", _out);
    fflush(stdout);

    FILE * sstream = fdopen(_args->sockfd, "w+");
    fprintf(sstream, "Message was: %s\n", _out);
    fflush(sstream);
    shutdown(_args->sockfd, 2);
}

我认为问题出在该函数末尾的某个地方，因为第二个 telnet 客户端没有收到“消息已发送:”行。

任何帮助将不胜感激!干杯。

Answer 1

这是因为 undefined behavior ，并且您的线程函数中有这个 UB，源于以下几行:

thrpass_st * _args = (thrpass_st *) malloc(sizeof(thrpass_st));
_args = (thrpass_st *) args;

第一行分配内存，但随后您用另一个指针覆盖指向该内存的指针，该指针指向另一个函数内的局部变量，该变量一旦 accept 就超出范围。循环迭代。

最简单的解决方案是复制结构:

*_args = *(thrpass_st *) args;

<小时/>

实际上，在这种情况下根本不需要指针，只需执行例如即可。

thrpass_st _args = *(thrpass_st *) args;

这样也不会因为忘记free而出现内存泄漏。线程末尾的指针。

<小时/>

另请注意，这里有一个竞争条件。如果两个客户端连接非常紧密，则存在两个线程将具有作为参数传递的相同结构数据的风险。

正确的解决方案是为循环中的参数结构分配内存，并将该指针传递给线程函数。并且不要忘记free当然是线程函数末尾的内存。