Discussion :

[julian75]

Bonjour,

Je dois écrire un client HTTP en utilisant les sockets et les appels systèmes Unix.
J'ai bien démarrer, mais je me retrouve devant un problème de lecture des données.
Voici le code et quelques explications ensuite :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
	int s = socket(AF_INET, SOCK_STREAM, 0);
 
	struct hostent *host = gethostbyname("www.perdu.com");
	struct sockaddr_in address;
	address.sin_family = AF_INET;
	address.sin_port = htons(80);
	address.sin_addr.s_addr = *(long *) host->h_addr_list[0];
 
	if (connect(s, (struct sockaddr*)&address, sizeof(address)) < 0) {
		perror("Socket connection");
		exit(2);
	}
 
	// Send HTTP Query
	char *httpQuery = "GET /index.html http/1.1 \nHOST: www.perdu.com \n\n";
	if (write(s, httpQuery, sizeof(httpQuery)/*, 0*/) < 0) {
		perror("HTTP Query");
		exit(3);
	}
	printf("query sent : \n%s \n", httpQuery);
	// Read content
	int l;
	char buffer[50];
	l = read(s, buffer, sizeof(buffer)/*, 0*/);
	printf("received = %s \n", buffer);

Le souci est qu'il bloque sur le read sans rien faire et sans rien afficher. Je dois zapper quelque chose car j'ai fais pas mal de recherche sans succès...
J'ai également testé avec les fonctions send() et recv() en mettant l'argument flags à 0.

Do you have any idea ?

Merci !

Ju

[ram-0000]

Ton send n'est pas bon.

Le 3eme paramètre (sizeof(httpQuery)) vaut 4 car httpQuery est un char *.

Il faut utiliser strlen à la place de sizeof pour transmettre la longueur de ta chaine. Et comme le send n'est pas bon, tu ne reçois rien.

C'est le genre de truc que tu aurais vu immédiatement si tu avais débuggué avec un analyse réseau (Wireshark par exemple)

[orfix]

Tu peux aussi écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char httpQuery[] = "GET /index.html http/1.1 \nHOST: www.perdu.com \n\n";

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char buffer[50];

Ce buffer est trop petit aussi !

[julian75]

Merci beaucoup messieurs !

Pour la longueur du buffer oui je vais l'augmenter mais là c'était vraiment pour tester un envoi-réception.

Encore merci !

Ju

[readln]

Tu peux aussi écrire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char httpQuery[] = "GET /index.html http/1.1 \nHOST: www.perdu.com \n\n";

Youpiiiiiiii pour faire plein de tout petits buffers overflows si je lui code une page truquée qu'il visitera avec son client HTTP

[orfix]

@readln: que veux-tu dire exactement ? un exemple peut-être ?

[readln]

@readln: que veux-tu dire exactement ? un exemple peut-être ?

Une chaine de caractére de taille indéfinie remplie directement depuis une requete, sans aucun controle, je te parie ce que tu veux que j'arrive a lui injecter un vilain code dans une page truquée.

Pour un exemple, je devrais voir l'implémentation compléte

[orfix]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char httpQuery[] = "GET /index.html http/1.1 \nHOST: www.perdu.com \n\n";

nous parlons bien de cette même ligne de code ?

Une chaine de caractére de taille indéfinie ...

Sa taille n'est pas indéfinie, elle est de sizeof httpQuery.

...remplie directement depuis une requete, sans aucun controle, je te parie ce que tu veux que j'arrive a lui injecter un vilain code dans une page truquée.

Quel genre de contrôle ?