服务器认为 2 个不同 IP 的客户端是同一个客户端（C 套接字编程）

Question

我有一台服务器，它应该允许一个客户端一次连接、执行命令然后关闭。服务器保留“当前连接的”IP 地址的数组列表，以便知道从谁那里接受命令。 问题是我有两个运行在相似 IP 地址上的客户端被视为同一个客户端。

示例： 客户端 A 的 IP 地址为 255.255.255.153。我告诉它连接，一切正常。 客户端 B 的 IP 地址为 255.255.255.156。我告诉它连接，服务器说它已经连接。

我的 strcmp 不正确吗？是别的吗？请帮忙。

// Variable Declarations

int SIZE = 10; // Max number of agents
char *agents[SIZE], // List of current connections
     *times[SIZE]; // List of connection times
char buffer[MAXBUF];
int bytes_read = 0;
int total_bytes_read = 0;
int found = 0, // Bool flag
    i;
struct tm  ts;
struct timeval connected[SIZE],
               current,
               difference;
       time_t TIME;

// Initialize array to NULL
for(i = 0; i < SIZE; i++) {
    agents[i] =  NULL;
}

// Infinite Loop
while (true) {
    struct sockaddr_in client;
    int clientSocket;
    socklen_t clientLength = sizeof(client);
    memset(&client, 0, clientLength);
    clientSocket = accept(sd, (struct sockaddr *)&client, &clientLength);

    memset(buffer,0,MAXBUF);
    bytes_read = read(clientSocket, buffer, MAXBUF);
    if (bytes_read < 0)
        break;
    fprintf(stdout,"\nRead %d bytes: [%s]\r\n", bytes_read,buffer);

    char *connectedIP = inet_ntoa(client.sin_addr);

    // Option 1:
    if ((strcmp(buffer, "#JOIN")) == 0) { // Join list of connected agents
        found = 0;

        // Get current time for log
        TIME = time(NULL);
        ts = *localtime(&TIME);

        // Print message to log
        char buf[80];
        strftime(buf, sizeof(buf), "%a %Y-%m-%d %H:%M:%S %Z", &ts);
        fprintf(log, "%s: Received a \"#JOIN\" action from agent \"%s\"", buf, connectedIP);
        fprintf(log, "\n\n");
        fflush(log);

        printf("\n%s Joining", connectedIP);
        fflush(stdout);
        // Handle #JOIN request appropriately
        for (i = 0; i < SIZE; i++) {
            printf("\nAgent[%d] == %s", i, agents[i]);

            if (agents[i] == NULL) {

            }
            else if (strcmp(agents[i], connectedIP) == 0){ // Agent found in list
                found = 1;

                printf("\n%s is equal to %s", agents[i], connectedIP);
                fflush(stdout);

                // Write to agent
                char response[] = "#ALREADY MEMBER";
                write (clientSocket, response, strlen(response));

                // Get time for log
                TIME = time(NULL);
                ts = *localtime(&TIME);

                // Write to log
                char buf[80];
                strftime(buf, sizeof(buf), "%a %Y-%m-%d %H:%M:%S %Z", &ts);
                fprintf(log, "%s: Responded to agent \"%s\" with \"#ALREADY MEMBER\"", buf, connectedIP);
                fprintf(log, "\n\n");
                fflush(log);
                i = SIZE;
            }
        }
        if (found == 0) { // Save IP to list/queue
            for(i = 0; i < SIZE; i++) {
                if (agents[i] == NULL) {

                    // Save IP to array
                    agents[i] = connectedIP;

                    printf("\nagents[%d] = %s\n", i, connectedIP);
                    fflush(stdout);

                    // Save time to arrays
                    gettimeofday(&connected[i], NULL);
                    TIME = time(NULL);
                    times[i] = TIME;
                    ts = *localtime(&TIME);

                    // Write to log
                    char buf[80];
                    strftime(buf, sizeof(buf), "%a %Y-%m-%d %H:%M:%S %Z", &ts);
                    fprintf(log, "%s: Responded to agent \"%s\" with \"#OK\"", buf, connectedIP);
                    fprintf(log, "\n\n");
                    fflush(log);

                    // Write to agent
                    char response[] = "#OK";
                    write (clientSocket, response, strlen(response));
                    i = SIZE;
                }
            }
        }
        memset(&client, 0, clientLength);
    }

Answer 1

inet_ntoa() 的 POSIX 规范包括以下信息：

inet_ntoa() 函数应将 in 指定的 Internet 主机地址转换为 Internet 标准点表示法中的字符串。

inet_ntoa() 函数不必是线程安全的。

inet_ntoa() 的 Linux 手册页更明确地说明了它的行为方式以及 POSIX 警告在该平台上的含义：

inet_ntoa() 函数将以网络字节顺序给出的 Internet 主机地址转换为 IPv4 点分十进制表示法的字符串。该字符串在静态分配的缓冲区中返回，后续调用将覆盖该缓冲区。

正如我在comment 中指出的那样（是的，该评论的第一部分并不准确——iPhone 应用程序上的代码很难阅读）：

...但是您可能会为所有代理使用相同的空间，因此如果您遍历连接的代理，打印名称，您会发现它们都是相同的。我希望那里有 strdup(connectedIP) 之类的东西。

这意味着由于您的代码只是将inet_ntoa() 返回的指针保存在agents 数组中，因此该数组始终包含最后查找的主机。您必须从inet_ntoa() 获取结果副本并保存，并管理存储。最简单的方法是使用strdup() 进行复制，但您必须记住在连接关闭时释放它。

使用strdup() 或等效机制应该可以解决您的直接问题。小心内存泄漏。