pthread_join не влияет на основной поток

Я нашел этот учебник по сокетам http://www.binarytides.com/socket-programming-c-linux-tutorial/ и у меня возникли проблемы с последним примером. Это многопоточный сервер, использующий сокеты и pthreads.

Код компилируется нормально, но работает не так, как ожидалось. Я подозреваю, что виноват звонок pthread_join, но я не уверен.

Я пытался скомпилировать и запустить сервер и клиент как на Cygwin (32-разрядная версия), так и на Alpine Linux (32-разрядная версия) с GCC 4.8.3 и 4.8.2. Проблема одинакова в обеих средах.

Я прочитал документацию по pthreads pthread_join по адресу http://pubs.opengroup.org/onlinepubs/9699919799/functions/pthread_join.html, но я не могу найти ничего плохого в вызове pthread_join в коде сервера.

Обновлен сервер, клиент и вывод.

Код сервера:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <sys/socket.h>
#include <arpa/inet.h>
#include <unistd.h>
#include <pthread.h>

void *connection_handler(void *);

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

    // 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(2000);

    // Bind
    if (bind(socket_desc, (struct sockaddr *) &server, sizeof(server)) < 0)
    {
        puts("bind failed");
        return 1;
    }
    puts("bind done");

    // Listen
    listen(socket_desc, 3);

    // Accept and incoming connection
    puts("Waiting for incoming connections...");
    c = sizeof(struct sockaddr_in);

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

        // Reply to the client
        message = "Hello Client, I have received your connection. And now I will assign a handler for you\n";
        write(new_socket, message, strlen(message));

        pthread_t sniffer_thread;

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

        // Now join the thread , so that we don't terminate before the thread
        // pthread_join(sniffer_thread, NULL);
        puts("Handler assigned");
    }

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

/*
 * This will handle connection for each client
 * */
void *connection_handler(void *socket_desc)
{
    // Get socket descriptor
    int sock = (int)socket_desc;
    int read_size;
    char *message, client_message[2000]; 

    // Send some messages to the client
    message = "Greetings! I am your connection handler\n";
    write(sock, message, strlen(message) + 1);

    message = "Now type something and i shall repeat what you type\n";
    write(sock, message, strlen(message) + 1);

    // Receive a message from client
    read_size = recv(sock, client_message, 1999, 0); 

    if (read_size > 0) 
    {
        client_message[read_size] = 0; 
    }
    else
    {
        puts("recv failed");
    }

    //while(read_size > 0 )
    //{
        // Send the message back to client
        write(sock, client_message, strlen(client_message) + 1);
    //}

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

    close(sock);

    return 0;
}

Код клиента:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/socket.h>
#include <arpa/inet.h>

int main(int arc, char *argv[])
{
    int socket_desc;
    struct sockaddr_in server;
    char *message, server_reply[2000];

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

    server.sin_addr.s_addr = inet_addr("127.0.0.1");
    server.sin_family = AF_INET;
    server.sin_port = htons(2000);

    // Connect to remote server
    if (connect(socket_desc, (struct sockaddr *) &server, sizeof(server)) < 0)
    {
        puts("connect error");
        return 1;
    }

    puts("Connected\n");

    // Send some data
    message = "Hello server!";
    if (send(socket_desc, message, strlen(message), 0) < 0)
    {
        puts("Send failed");
        return 1;
    }
    puts("Data Send\n");

    int length = 0;
    do {
        // Receive a reply from the server
        length = recv(socket_desc, server_reply, 1999, 0); 

        if (length > 0)
        {
            server_reply[length] = 0; 
        }
        else
        {
            puts("Nothing to recieve");
            close(socket_desc);
            return 1;
        }

        puts("Reply received\n");
        puts(server_reply);

    } while (length > 0);

    return 0;
}

Вывод с сервера:

bind done
Waiting for incoming connections...
Connection accepted
Test 1
Segmentation fault (core dumped)

Вывод от клиента:

Connected
Data Send
Reply received
Hello Client , I have received your connection. And now I will assign a handler for you
recv failed

Похоже, что основной поток возвращается/завершается раньше, чем sniffer_thread.

Любая помощь приветствуется.


person user1359448    schedule 30.09.2014    source источник
comment
Первое, что я делаю с сетевым кодом C, — это ищу вызовы strlen(). Разумеется, 'strlen(client_message)' вызывается для буфера, который не завершается нулем.   -  person Martin James    schedule 30.09.2014
comment
Возьмите значение, возвращаемое вызовом recv() (что-то, что вы по существу игнорируете ATM), и используйте его, чтобы вставить нуль в конец полученных данных. Это остановит strlen() от segfaulting, запустив конец буфера. Однако это, вероятно, не единственная проблема.   -  person Martin James    schedule 30.09.2014
comment
'кладет (server_reply);' имеет ту же проблему на клиенте - ему нужен нуль-терминатор, а буфер автоматического хранения server_reply не гарантирует, что он будет в конце данных.   -  person Martin James    schedule 30.09.2014
comment
Если вы хотите отправить нуль-терминатор на «строки», строки типа «write(sock, message, strlen(message));» не отправит его - вам нужно добавить 1 к int, возвращаемому функцией strlen().   -  person Martin James    schedule 30.09.2014
comment
Всегда инициализируйте все переменные. Всегда и всегда (по крайней мере, на этапе отладки) проверяйте все системные вызовы на наличие всех возможных значений/диапазонов, которые они могут вернуть.   -  person alk    schedule 30.09.2014
comment
Кстати - не расстраивайтесь из-за этого. 99% всего сетевого кода C, размещенного на SO, имеет проблемы с нулевым завершением strlen/printf/puts и т. д. и игнорированием результатов системных вызовов (особенно recv).   -  person Martin James    schedule 30.09.2014
comment
Хорошо, я только что посмотрел примеры серверов/клиентов на сайте. Удалите ссылку из истории, закладок и мозга. Никогда не смотрите на это снова.   -  person Martin James    schedule 30.09.2014
comment
О, и избавьтесь пока от 'pthread_join' - просто закомментируйте его. Это остановит цикл приема сервером от принятия каких-либо клиентов. Цикл while останавливает завершение вызывающего его потока (то есть того, который запускает main).   -  person Martin James    schedule 30.09.2014
comment
Ха-ха, заметил. я внедрил предложенные вами изменения, но я не понимаю, что касается добавления нуля в конце полученных данных, recv возвращает длину полученных данных.   -  person user1359448    schedule 30.09.2014
comment
@ user1359448 - да, и это позиция в буфере, куда должен идти нуль :) Не нужно сначала обнулять весь буфер 2 КБ, просто поместите ноль в конец того, что было получено - как возвращено recv (при условии, что не -1 (ошибка, в этом случае получите errno и распечатайте ее) или 0 (соединение закрыто удаленным узлом — ваш сигнал к очистке и выходу из потока).   -  person Martin James    schedule 30.09.2014
comment
@user, но recv не вставляет автоматически '\ 0' в конец прочитанных данных   -  person 4pie0    schedule 30.09.2014
comment
int length=recv(socket_desc, server_reply, 1999, 0); if(length›0) server_reply[length]=0 else {обработка ошибок};   -  person Martin James    schedule 30.09.2014
comment
Примечание. '1999', чтобы оставить место для нуля, если получено 1999 символов.   -  person Martin James    schedule 30.09.2014
comment
@Martin James Спасибо за вашу большую помощь, я думаю, что внес все изменения в код, который вы предложили, и я обновил код и вывод выше, но я все еще получаю то же самое поведение, что и раньше. Значит, я должен делать что-то не так.   -  person user1359448    schedule 30.09.2014
comment
Хорошо, теперь о других вещах. выделение одного байта недостаточно для хранения типичного дескриптора сокета handle/fd : «В случае успеха эти системные вызовы возвращают неотрицательное целое число, которое является дескриптором принятого сокета». Вам нужно 4/8 байт для его хранения. Как правило, void* имеет тот же размер, что и int, поэтому вы, вероятно, могли бы просто привести его: '(void*)new_socket' для вызова pthread_create и просто полностью сбросить malloc. На данный момент он работает через UB.   -  person Martin James    schedule 01.10.2014
comment
«Тест 3» требует «strlen(message)+1», как и тест 2. Кроме того, нет необходимости добавлять явный «\0» в конце литеральных строк.   -  person Martin James    schedule 01.10.2014
comment
В тесте 4 избавьтесь от «strlen(client_message)» и замените его на «read_size».   -  person Martin James    schedule 01.10.2014
comment
'бесплатно (socket_desc);' - Хорошо, потому что вы изначально выделили его, но вы также должны закрыть «sock», так как это ресурс ОС, возвращаемый вызовом accept().   -  person Martin James    schedule 01.10.2014
comment
Хорошо, теперь о клиенте. В нем отсутствует цикл while, поэтому recv() можно вызывать более одного раза, поэтому он просто возвращается после получения первых данных «Hello Client...».   -  person Martin James    schedule 01.10.2014
comment
В 'помещает (сбой получения);' блок, вам нужно закрыть socket_desc и выйти из цикла (того, которого у вас еще нет :).   -  person Martin James    schedule 01.10.2014
comment
@MartinJames Я реализовал все изменения, которые вы предложили на сервере и клиенте, многие из них имеют смысл;) Но теперь я получаю segfault на сервере. Я обновил клиент, серверный код выше и вывод обоих. Спасибо за вашу помощь   -  person user1359448    schedule 01.10.2014
comment
лол — Хорошо, вы приняли мое предложение передать new_socket непосредственно в вызове pthread_create посредством приведения его к void , но вы не исправили другой конец — поток все еще думает, что это указатель с распределенным адресом. Вам нужно исправить 'int sock = *(int)socket_desc;' просто возвращаясь к типу int: 'int sock=int(socket_desc);'. . Кроме того, вам больше не нужно освобождать его при выходе из потока, просто закройте его.   -  person Martin James    schedule 01.10.2014
comment
@MartinJames Я обновил код сервера и клиента, и теперь он работает. Вы скомпилируете свои комментарии в ответ? :)   -  person user1359448    schedule 08.10.2014


Ответы (1)


Вы испытываете неопределенное поведение, потому что вы вызываете strlen для строки, которая не заканчивается символом '\0'. recv не помещает нулевой терминатор в конец строки.

char *message , client_message[2000];
//...
//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));
                                     ^^^^^^^^^^^^^^
                                     not null terminated

Вы должны использовать возвращаемое значение recv для проверки размера прочитанных данных.

person 4pie0    schedule 30.09.2014
comment
Неправильно. Литеральная строка имеет нуль-терминатор. Проблема в том, что а) он не будет отправлен узлу и б) даже если он будет отправлен, он не может быть прочитан одним вызовом recv() на другом конце. - person Martin James; 30.09.2014
comment
@MartinJames исправил, я имел в виду вызов strlen(client_message) спасибо - person 4pie0; 30.09.2014