Web server in c

[Paniko84]

ciao a tutti,

sto scrivendo un semplicissimo web server in c sotto linux (ubuntu 6.10). Ho creato la socket sulla quale il server instaura la connessione TCP con chi vuole fare la richiesta HTTP. Una volta instaurata la connessione il server crea un thread che si occupa di soddisfare quella richiesta (a cui passa la socket sulla quale comicare). Il thread quindi fa la read() e stampa ciò che ha ricevuto.

Quello che ottengo stampato a schermo pero' è questo

4��G,�� ��(H��

a occhio me sembrerebbe che non sto stampando una stringa ma un riferimento a memoria pero' non capisco perchè.
In più volevo chiedervi come faccio a gestire una richiesta HTTP. DEvo scrivere le funzioni per fare il parsing della richiesta di mio pugno o ci sono delle librerie già pronte???

allego il codice


#include <stdio.h> /* per usare input-output */
#include <stdlib.h> /* per EXIT_FAILURE */
#include <sys/socket.h> /* per usare socket */
#include <sys/types.h>
#include <netinet/in.h>
#include <netdb.h>
#include <errno.h> /* gestione degli errori */
#include <pthread.h>
#include <linux/unistd.h>

int sock; /* socket di ascolto, globale , è sempre la stessa */

void *gestore_richiesta(void *arg) {
int sockmsg=*((int*)arg);
printf("thread %d: sono stato creato\n", pthread_self() );
//close(sock); //se i thread condividono la stessa tabella dei descrittori cosi chiud anche il server
//printf("thread %d: ho chiuso la socket di ascolto e lascio aperta quella di dialogo\n", pthread_self() );

char input[256];
int len;

len = read(sockmsg,input,sizeof(input));
input[len]='\0';

printf("len: %d msg: %s",len,input);

close(sockmsg);
printf("thread %d: ho chiuso la socket di dialogo\n", pthread_self() );
}

main(int argc, char** argv)
{

//--- DICHIARAZIONI----

struct sockaddr_in server;
int sockmsg; /* socket di dialogo */

//---VERIFICA DELLA CORRETTEZZA DELL'INPUT---

if ( argc != 2 ) {
printf("uso: %s <numero-della-porta>\n",argv[0]);
exit(EXIT_FAILURE);
}

//CHIEDE AL SO UNA SOCKET

sock = socket(AF_INET, SOCK_STREAM, 0);

if( sock < 0 ) {
printf("server %d: errore %s nel creare la socket\n", getpid(), strerror(errno));
exit(1);
}

server.sin_family = AF_INET;
server.sin_addr.s_addr = INADDR_ANY;
server.sin_port = htons(atoi(argv[1]));

if( bind(sock, (struct sockaddr *)&server, sizeof(server)) ) {
printf("server %d: bind fallita\n", getpid());
exit(EXIT_FAILURE);
}


if( listen(sock, 4) <0 ) {
printf("server %d: errore %s nella listen\n",getpid(), strerror(errno));
exit(EXIT_FAILURE);
}

printf("server %d: sono in ascolto sulla porta %d\n", getpid(), ntohs(server.sin_port));


while(1) {



if( (sockmsg = accept(sock, 0, 0)) <0 ) {
printf("server %d: errore %s nella accept\n",getpid(), strerror(errno));
exit(1);
}
printf("server %d: accettata una nuova connessione\n", getpid());


pthread_t mythread;
int i;

//CREO IL THREAD CHE GESTISCE LA RICHIESTA

if (pthread_create(&mythread,NULL,gestore_richiesta,&sockmsg)) {
printf("server %d: creazione del thread fallita.",getpid());
exit(1);
}


close(sockmsg);
printf("server %d: ho chiuso la socket di dialogo\n",getpid());

}

close(sock); /* inutile: non arrivero’ mai qui! */
printf("server %d: ho chiuso la socket di ascolto\n",getpid());
}


l'output del programma invece è questo:


***********-laptop:/media/hda6/svil/pc$ ./webserver 8084
server 26337: sono in ascolto sulla porta 8084
server 26337: accettata una nuova connessione
server 26337: ho chiuso la socket di dialogo
thread -1209660512: sono stato creato //qui ho fatto una richiesta usando mozilla
len: 16777215 msg: 4��G,�� ��(H��thread -1209660512: ho chiuso la socket di dialogo

A questo punto il server giustamente continua a essere in attesa di altre richieste
Se faccio altre richieste non viene stampato nulla. Forse ho messo troppa carne al fuoco, se qualcuno trova il tempo e la pazienza di darmi una mano mi tira fuori dagli impicci :P
grazie cmq a tutti anticipatamente

[ilsensine]

len = read(sockmsg,input,sizeof(input));

Il valore di ritorno chi lo controlla?

input[len]='\0';

Buffer overflow se len=256. Metti nella read sizeof(input)-1.

if (pthread_create(&mythread,NULL,gestore_richiesta,&sockmsg))

Race grossolana. Passi il _puntatore_ a sockmsg al secondo thread, ma sockmsg è una variabile del primo thread che può venire modificata senza preavviso!

close(sockmsg);

Ecco l'origine dei tuoi guai. I thread condividono la tabella degli fd!


len: 16777215 msg: 4��G,�� ��(H��thread -1209660512: ho chiuso la

19777215 vale 0x00ffffff. Dieci a uno che la read ha ritornato -1 a causa della close(sockmsg), e tu hai scritto input[-1] = '\0'.

[Paniko84]

grazie mille per la prontissima risposta !!!!


19777215 vale 0x00ffffff. Dieci a uno che la read ha ritornato -1 a causa della close(sockmsg), e tu hai scritto input[-1] = '\0'.


Meno male che non si era parlato di soldi senno andavo fallito :D
... effettivamente l'errore del passaggio del puntatore era un po ingenuo. E' la prima volta che uso i thread e mi era sfuggito che effettivamente la tavola dei descrittori è associata al processo e quindi condivisa tra tutti i thread del processo...

adesso funziona correttamente...ecco alcune richieste stampate sul terminale :D grazie ancora



jimmi@jimmi-laptop:/media/hda6/svil/pc$ ./webserver 8084
server 1557: creata la socket di ascolto: 3
server 1557: sono in ascolto sulla porta 8084
server 1557: accettata una nuova connessione
thread -1210057824: sono stato creato
skt: 4 len: 255 msg: GET /pippo.html HTTP/1.1
Host: 127.0.0.1:8084
User-Agent: Mozilla/5.0 (X11; U; Linux i686; it; rv:1.8.1.4) Gecko/20060601 Firefox/2.0.0.4 (Ubuntu-edgy)
Accept: text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/
thread -1210057824: ho chiuso la socket di dialogo
server 1557: accettata una nuova connessione
thread -1218450528: sono stato creato
skt: 5 len: 255 msg: GET /pippo.html HTTP/1.1
Host: 127.0.0.1:8084
User-Agent: Mozilla/5.0 (X11; U; Linux i686; it; rv:1.8.1.4) Gecko/20060601 Firefox/2.0.0.4 (Ubuntu-edgy)
Accept: text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/
thread -1218450528: ho chiuso la socket di dialogo



ora non mi resta fare il parsing della richiesta HTTP prendere il file e restituirlo al client.
Non so se esistono delle librerie apposta per gestire le richieste HTTP e fornire risposte, ho cercato con google ma non sono riuscito a trovare nulla. Per caso conosci qualcosa? Senno risolvo scrivendo delle funzioni che elaborano la stringa e generano la risposta HTTP di tipo POST ... inizio a lavorarci su ;)