【问题标题】：Reading client response crashes server读取客户端响应使服务器崩溃
【发布时间】：2017-09-30 03:28:51
【问题描述】：

我正在编写一个程序，其中服务器由另一个程序发出信号，在收到信号后，它从共享内存段读取目录名称并通过 sock 将其发送给客户端。客户端发回目录内容并断开连接

我的问题是，当服务器收到客户端响应时，它会打印出来并立即停止监听端口。

该程序适用于所有其他情况，例如客户端连接，不发送任何内容，然后断开连接。

void handler(int signal_number)
{   
    int read_size;
    char* cli_dir[1000];
    char *message , client_message[2000];
    int i = 0;

    printf("about to access shared memory");
    message = shm; //shm is the directory name in shared memory
    puts("accessed shared memory");
    printf("is gonna be sent to client %s\n",message);
    write(sock , (char*)message , strlen(message));

    while((read_size = recv(sock , client_message , 2000 , 0)) > 0 )
    {
        //read client response and log contents of the received directory
        printf("\n%s\n", (char*)client_message);
        strcpy(cli_dir[i],(char*)client_message);
        i++;
        printf("\n");
        perror("error here");
    }

    if(read_size == 0)
    {
        puts("Client disconnected");
        fflush(stdout);
        perror("error 2 here"); 
    }
    else if(read_size == -1)
    {
        perror("receiving server side failed");
    }
}

void* connection_handler(void *socket_desc)
{
    //Get the socket descriptor
    printf("accessed connection handler");

    sock = *(int*)socket_desc;

    while(1){

        signal(SIGUSR1, handler);

        // struct sigaction sa;
        // printf("signal recieved");
        // memset(&sa, 0, sizeof(sa)); //alternative signal handler
        // sa.sa_handler = &handler;
        // sigaction(SIGUSR1, &sa, NULL);

        pause();
    }
}

另外，这是我在主函数中接受连接的方式

    while( (client_socket = accept(socket_desc, (struct sockaddr *)&client, (socklen_t*)&c)) )
    {
        puts("Client accepted");

        pthread_t sniff;
        new_sock = malloc(1);
        *new_sock = client_sock;

        if( pthread_create( &sniffer_thread , NULL ,  connection_handler , (void*) new_sock) < 0)
        {
            perror("could not create thread");
            return 1;
        }

        pthread_join( sniff , NULL);
        puts("client handled");
    }

    if (client_socket < 0)
    {
        perror("accept failed");
        return 1;
    }

【问题讨论】：

您应该发布足够多的代码，以便有人可以编译......就目前而言，任何人都不能真正“测试”
'printf("\n%s\n", (char*)client_message);'需要一个以 NUL 结尾的字符数组，这不是您的代码所支持的。
...和'strcpy(cli_dir[i],(char*)client_message);'
'pthread_join（嗅探，NULL）；'调用未初始化的变量。
'pthread_join（嗅探，NULL）；'即使它有效，也不需要并阻止接受循环。

标签： c linux server signals posix

【解决方案1】：

首先，正如马丁詹姆斯已经说过的，

printf("\n%s\n", (char*)client_message);

需要一个以 NUL 结尾的字符串。这不是由您的服务器代码强制执行的，如果您从像

这样的客户端发送一个以 NUL 结尾的字符串

write(fd, str, strlen(str));

那么您将不会发送'\0'（您需要发送strlen(str) + 1 字节或在接收端添加'\0'，注意不要让缓冲区溢出1 个字节）。

其次，我不熟悉编写信号安全/不安全代码，因此无法对此发表评论，但这里也可能存在问题。

主要问题

恕我直言，您的代码的主要问题是您声明了一个指向字符串的指针的未初始化数组cli_dir，然后将此类指针传递给strcpy：

strcpy(cli_dir[i],(char*)client_message);

如果我的代码正确，在这里你传递一个指向内存位置的指针，strcpy 必须将一个字符串复制到该位置，但你没有以某种方式为副本分配内存，cli_dir[i] 现在包含一些任意的垃圾。这很可能会导致分段错误。

【讨论】：