它如何成为一种比赛条件。或者我的代码有问题

Question

我认为我面临着比赛条件的问题，但不确定。以下是我的代码。

while(1) 
{
    int newsocket_fd; = accept(socket_fd,(struct sockaddr *) &client_addr, &client_len);

    if (newsocket_fd < 0) 
    {
          perror("Error on accept");
    }
    else
    {
        pthread_create( &threadNewClient, NULL, NewClient, (void *)&newsocket_fd);
    }
}

我将int变量（newsocket_fd）的地址作为线程参数发送。

void* NewClient(void * thread_args)
{
    int client_socket = *(int*)thread_args;
    char buffer[256];   
    int n;
    stringstream userInput,responseString; 

    printf(KBLU "Client conneted  %d\n",client_socket);
}

在这里，我接收线程参数并对其进行类型转换。当我打印(或使用)它的值时，它大部分时间给出发送给以前线程的值(同一个线程函数)。我在网上搜索，有人认为这是因为种族条件。

当我发送值而不是地址作为参数时，它可以正常工作。我正在访问具有不同值的同一地址，这可能是竞争条件。

先谢谢你。

@编辑：创建intnewsocket_fd；在while循环内。

Answer 1

newsocket_fd可能会在新线程从你给它的指针中读取它之前被修改（或者更糟，被销毁并重新创建）。这绝对是一个竞争条件。

最简单的解决方法就是线程之间不共享相同的变量。如果出于某种原因，您绝对必须这样做，那么您必须确保在线程使用某种同步对象（条件）读取它之前，它不会被修改。

Answer 2

因为您正在发送一个位于堆栈的整数（newsocket_fd）的地址。这将在下次接受套接字时被重用。只发送整数值有什么问题？