0

我正在尝试掌握多线程编程的概念，并且我认为我做得很好，但是后来我为一个简单的echo服务器找到了以下代码：

http://www.cs.utah.edu/~swalton/listings/sockets/programs/part2/chap7/echo-thread.c

而且我认为代码是错误的，因为它使用相同的main局部变量来存储每个传入连接的数据套接字。特别是，我担心这部分main()：

while (1)
{   int client, addr_size = sizeof(addr);
    pthread_t child;

    client = accept(sd, (struct sockaddr*)&addr, &addr_size);
    printf("Connected: %s:%d\n", inet_ntoa(addr.sin_addr), ntohs(addr.sin_port));
    if ( pthread_create(&child, NULL, Child, &client) != 0 )
        perror("Thread creation");
    else
        pthread_detach(child);  /* disassociate from parent */
}

据我了解，循环client本地的变量在while循环的每次迭代中都分配在完全相同的地址。因此，当第一个客户端被接受时，线程接收&client，当第二个客户端被接受时，client新的数据套接字会覆盖 的值，这会对已经在第一个客户端上运行的线程产生副作用.

观察函数的代码Child，也就是服务线程，我可以看到参数被复制到了一个局部变量中：

void* Child(void* arg)
{   char line[100];
    int bytes_read;
    int client = *(int *)arg;
    ...etc...

并且可能作者认为这个副本允许他稍后篡改主client变量，但恕我直言，这可能会导致竞争条件。如果第二个客户端在第一个线程复制此变量时到达，则复制的值可能已损坏。

我对吗？

4

2 回答 2

1

是的，你是对的。有两种明显的方法可以解决这个问题：

  1. 传递client给线程而不是&client.

  2. 在堆上分配一个新的整数并将其地址传递给线程，并在完成后让线程释放它。

于 2013-02-10T22:11:54.513 回答
1

是的你是对的。

sleep您可以通过在第一个客户端线程时添加一个 long beforeint client = *(int*)arg;并连接到服务器两次来证明您是正确的。

于 2013-02-10T22:13:35.457 回答