Несколько клиентов не подключены и не могут общаться одновременно

Question

Ниже приведен код сервера, который я написал для нескольких клиентов. Но если я хочу подключить его ко второму клиенту, я не могу подключить его одновременно. Сначала мне нужно закрыть первый клиент, а затем только я могу подключиться и связаться со вторым клиентом. Я думаю, что у меня есть некоторые проблемы при использовании pthread_join, Не уверен, что именно проблема. Я хочу, чтобы сервер работал для нескольких клиентов одновременно.

#include<stdio.h>
#include<string.h>    //strlen
#include<stdlib.h>    //strlen
#include<sys/socket.h>
#include<arpa/inet.h> //inet_addr
#include<unistd.h>    //write
#include<pthread.h> //for thread


 #define MAX_CLIENTS 5


//the thread function
void *new_connection_handler(void *);

int main(int argc , char *argv[])
{
    int socket_desc , client_sock , c , *new_sock;
    struct sockaddr_in server , client;


    //Create socket
    socket_desc = socket(AF_INET , SOCK_STREAM , 0);
    if (socket_desc == -1)
    {
        printf("Could not create socket");
    }


    //Prepare the sockaddr_in structure
    server.sin_family = AF_INET;
    server.sin_addr.s_addr = INADDR_ANY;
    server.sin_port = htons( 8888 );
    bzero (&server.sin_zero, 8);



    //Bind
    if( bind(socket_desc,(struct sockaddr *)&server , sizeof(server)) < 0)
    {
        //print the error message
        perror("bind failed. Error");
        return 1;
    }


    //Listen
    listen(socket_desc , MAX_CLIENTS);



    //Accept and incoming connection
    printf("Waiting for incoming connections\n");




    c = sizeof(struct sockaddr_in);
    while( (client_sock = accept(socket_desc, (struct sockaddr *)&client, (socklen_t*)&c)) )
    {
        printf("Connection accepted");



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



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



     printf("Handler assigned\n");
    }


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

    return 0;
}


void *new_connection_handler(void *socket_desc)
{
    //Get the socket descriptor
    int sock = *(int*)socket_desc;
    int read_size;
    char *message , client_message[2000];



    //Send some messages to the client
    message = "This is connection handler\n";
    write(sock , message , strlen(message));


    message = "Type something \n";
    write(sock , message , strlen(message));


    //Receive a message from client
    while( (read_size = recv(sock , client_message , 2000 , 0)) > 0 )
    {
        //Send the message back to client
        write(sock , client_message , strlen(client_message));
    }



    if(read_size == 0)
    {
        printf("Client disconnected\n");
        fflush(stdout);
    }
    else if(read_size == -1)
    {
        perror("recv failed");
    }

    //Free the socket pointer
    free(socket_desc);

    return 0;
}

Answer 1

Нет необходимости объединять все темы. Отдельные темы подходят для этой задачи.

Я полагаю, вы действительно звоните pthread_join внутри while (accept(..)) цикл, иначе ваше описание не имеет особого смысла. Если это так, просто замените его pthread_detach,

Если вы хотите дождаться завершения всех потоков перед выходом, используйте условную переменную pthreads для подсчета активных потоков. В вашем случае основная программа никогда не завершается, поэтому вы можете просто проигнорировать проблему.