LInux中的C socket编程，双重免费损坏（fasttop）错误

-1

我已阅读其他类似问题的答案，但它们都不能解决我的问题。LInux中的C socket编程，双重免费损坏（fasttop）错误

这是我的代码：

#include <stdlib.h> 
#include <stdio.h> 
#include <string.h> 
#include <arpa/inet.h> 
#include <errno.h> 
#include <unistd.h> 
#include <sys/types.h> 
#include <sys/socket.h> 
#include <netdb.h> 
#include <netinet/in.h> 
#include <pthread.h> 

typedef struct { 
    pthread_t thread_id; 
    int sockfd; 
} client_t; 

client_t *clients; 
size_t client_n = 0; 

void *client_thread(void *client_ptr) { 
    client_t client = *(client_t*) client_ptr; 
    char buffer[500]; 
    int state; 
    while(1) { 
    state = send(client.sockfd, 0, 1, MSG_NOSIGNAL); 

    if(state == -1) { 
     printf("socket-%d closed\n", client.sockfd); 
     break; 
    } 

    read(client.sockfd, buffer, 500); 
    printf("from socket-%d: %s\n", client.sockfd, buffer); 
    memset(buffer, 0, 500); 
    } 
    close(client.sockfd); 
    free(client_ptr); 
    client_n--; 
} 

int main(int argc, char *argv[]) { 
    int sockfd, newsockfd, clilen; 
    struct sockaddr_in clientaddr, serveraddr; 

    sockfd = socket(AF_INET, SOCK_STREAM, 0); 
    serveraddr.sin_family = AF_INET; 
    serveraddr.sin_addr.s_addr = INADDR_ANY; 
    serveraddr.sin_port = htons(8080); 

    bind(sockfd, (struct sockaddr*) &serveraddr, sizeof(serveraddr)); 
    listen(sockfd, 5); 

    clilen = sizeof(clientaddr); 

    clients = (client_t*) malloc(sizeof(client_t)); 

    while(1) { 
    newsockfd = accept(sockfd, (struct sockaddr*) &clientaddr, &clilen); 
    printf("New connection: socket-%d\n", newsockfd); 
    clients = (client_t*) realloc(clients, (client_n + 1) * sizeof(client_t)); 
    clients[client_n].sockfd = newsockfd; 
    pthread_create(&clients[client_n].thread_id, NULL, client_thread, (void*) &clients[client_n]); 
    client_n++; 
    } 
    return 0; 
}

程序应该侦听传入的连接，然后创建为每一个新的线程。该程序将同时处理每个客户端。由于这应该是游戏服务器的核心，我创建了一个包含每个玩家信息的结构。

这一切都工作得很好，直到我加入：

close(client.sockfd); 
    free(client_ptr); 
    client_n--;

任何想法的问题是什么？

来源

2017-07-29 Francesco Cozzuto

您的分配已损坏。你分配一个'client_t'数组，但是尝试释放单个条目，这是不可能的。只需每次分配一个条目，将其分配给客户端线程，以便将其释放。 –

随着

free(client_ptr); 
client_n--;

有问题。

第一个是拨打free。你实际上没有拨malloc（或realloc或calloc）client_ptr。而是client_ptr指向您动态分配的数组，但client_ptr指向的元素本身并不是单独动态分配的。这会导致未定义的行为当您将指针传递给free时，实际上没有与malloc和系列一起分配。除了第一个元素（即clients[0]），当你改为释放整个数组。解决方法是在线程中不要调用free。

另一个问题是client_n--表达式。您不保护此操作（或main函数中的相应client_n++）不受其他线程修改。这意味着两个或多个线程可能会同时再次修改，导致未定义的行为。你需要有一个信号量或互斥量来保护这个修改。

还有一些其他问题。例如，您不加入已结束的线程，导致资源泄漏。您不检查来自read呼叫的错误或关闭连接（通过返回0的read呼叫报告一个很好的关闭连接）。

来源

2017-07-29 11:06:58

LInux中的C socket编程，双重免费损坏（fasttop）错误

回答

相关问题