2014-03-04 4 views
-1

Я пытаюсь изменить следующий код, который представляет собой простой многопоточный сокет-сервер, который, если клиент выдаст команду QUIT, тогда он фактически закроет соединение.Сервер сокета закрывает соединение для команды

/* 
    C socket server example, handles multiple clients using threads 
*/ 

#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 threading , link with lpthread 

//the thread function 
void *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"); 
    } 
    puts("Socket created"); 

    //Prepare the sockaddr_in structure 
    server.sin_family = AF_INET; 
    server.sin_addr.s_addr = INADDR_ANY; 
    server.sin_port = htons(8888); 

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

    //Listen 
    listen(socket_desc , 3); 

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


    //Accept and incoming connection 
    puts("Waiting for incoming connections..."); 
    c = sizeof(struct sockaddr_in); 
    while((client_sock = accept(socket_desc, (struct sockaddr *)&client, (socklen_t*)&c))) 
    { 
     puts("Connection accepted"); 

     pthread_t sniffer_thread; 
     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; 
     } 

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

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

    return 0; 
} 

/* 
* This will handle connection for each client 
* */ 
void *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 = "Greetings! I am your connection handler\n"; 
    write(sock , message , strlen(message)); 

    message = "Now type something and i shall repeat what you type \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) 
    { 
     puts("Client disconnected"); 
     fflush(stdout); 
    } 
    else if(read_size == -1) 
    { 
     perror("recv failed"); 
    } 

    //Free the socket pointer 
    free(socket_desc); 

    return 0; 
} 

То, что я сделал это, чтобы вставить разрыв в то время цикла в потоке обработчика соединения (который работает), то перед
бесплатно (socket_desc); Я добавил закрыть (socket_desc); , а также для закрытия соединения. Это создает ошибку компилятора, а также при сбое сервера при достижении этой точки.

com.c: В функции 'connection_handler': com.c: 134: 5: предупреждение: проезжают аргумент 1 из 'закрыть' делает целое число от указателя без приведения [по умолчанию включено]

+1

Я не вижу перерыва. Покажите свой реальный код, а не пример. – deviantfan

+0

Независимо от типа socket_desc, не может быть недействительным аргументом как для free(), так и для close(). Итак, что это - дескриптор файла 'int' или указатель на некоторую структуру информации? И -1 для размещения кучи кода, который не включает строку, вызывающую предупреждение компиляции, о котором вы спрашиваете. –

ответ

0

с (в вашем втором блоке кода), вы, кажется, использует sock_desc как void указатель на то, что вы знаете, на самом деле является int и sock как int полученного разыменовывая, что вы должны называть

close(sock); 

Как close(), как и другие функции сокетов, которые вы использовали, в качестве аргумента, а не указателя, вместо дескриптора используется дескриптор файла (который вы можете представить как слегка произвольный «номер файла», предоставляемый системой).

Смежные вопросы