Problème de Sockets

Version imprimable

Problème de Sockets

Salut !

Je fais actuellement un simple petit client serveur car je n'ai besoin d'envoyer que de petites informations d'un programme à l'autre et ce serait trop lourd d'utiliser un ORB pour faire cela.

J'ai donc décidé d'utiliser des sockets, et j'ai un tout peit problème que je ne comprend vraiment pas ... je lance mon serveur, et il attend la connexion ... je lance mon client et il me dit qu'il est connecté, mais mon serveur reste en attente de connexion et ne bouge pas de là ...

Voici mon code, est-ce que quelqu'un aurait une idée ?

Serveur :

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
#include <winsock2.h>
#include <iostream>
 
int main(int argc, char* argv[]){
 
  int attenteSocket;
  int acceptation;
  int sock;
  int bindS;
  int nb;
  char* clavier;
  int port;
  int lire;
  int i;
 
  char tampon[512];
 
  SOCKADDR_IN emetteur;
  SOCKADDR_IN hote;
  struct hostent* host;
 
  int longAddr;
 
  // Vérification du nombre de paramètres
  if(argc == 3){
    if (!strcmp(argv[1],"-port")) port = (int)argv[2];
	else port = 9999;
  } else {
    port = 9999;
  }
 
  WSADATA WSAData;
  WSAStartup(MAKEWORD(2,0), &WSAData);
 
  // Résolution DNS à partir du nom de la machine
  host = gethostbyname("localhost");
 
  //Initialisation de la structure sockaddr_in
  hote.sin_family = AF_INET;
  hote.sin_port = htons(port);
 
  memcpy(&hote.sin_addr,
	 host->h_addr,
	 host->h_length);
 
  for(int i=0; i<8; i++){
    hote.sin_zero[i] = 0;
  }  
 
  // Création du socket
  sock = socket(AF_INET, SOCK_STREAM, 0);
 
  // Test de la validité de la création du socket
  if(sock == -1){
    printf("Erreur à l'ouverture du socket\n");
    exit(-1);
  } 
 
  bindS = bind(sock, (SOCKADDR*)&hote, sizeof(hote));
 
  // Test de la validité du bind
  if(bindS == -1){
    printf("Erreur lors du bind du socket\n");
    exit(-1);
  } 
 
  printf("Socket créé et bindé\n");
 
  listen(sock,0);
  do{
    attenteSocket = listen(sock, 0);
  } while(attenteSocket == -1);
 
  printf("En attente de connexion\n");
 
  do{
    acceptation = accept(sock, (SOCKADDR*)&emetteur, &longAddr);
  } while(acceptation == -1);
 
  // Attente tant qu'on n'appuie pas sur la touche Espace
  printf("Appuyer sur espace pour lancer la consommation\n");
  do{
    scanf("%c",clavier);
  }while(!strcmp(clavier, " "));
 
  printf("Consomation lancée\n");
 
  while(1){
    lire = recv(acceptation, tampon, sizeof(tampon), 1024);
    if(lire != -1){
      printf("%s\n",i++,tampon);
    }
  }
 
}

Code client :

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
bool Acquisition::initConnexion(int argc , char* argv[]){
 
	bool res = true;
	WSADATA WSAData;
	WSAStartup(MAKEWORD(2,0), &WSAData);
 
	if(argc == 5){
		if(!strcmp(argv[1],"-host") && !strcmp(argv[3],"-port")){
			nomDest = argv[2];
			port = (int)argv[4];
		} else if(!strcmp(argv[3],"-host") && !strcmp(argv[1],"-port")){
			nomDest = argv[4];
			port = (int)argv[2];
		}
	} else if(argc == 3) {
		if(!strcmp(argv[1],"-host")){
			nomDest = argv[2];
			port = 9999;
		} else if(!strcmp(argv[1],"-port")){
			nomDest = "localhost";
			port = (int)argv[2];
		}
    } else if(argc == 1) {
		nomDest = "localhost";
		port = 9999;
	} else {
		usage(argv[0]);
		res = false;
	}
    int b = -10;
	if (res){
		// Résolution DNS à partir du nom de la machine
		host = gethostbyname(nomDest);
		//Initialisation de la structure sockaddr_in
		destinataire.sin_family = AF_INET;
		destinataire.sin_port = htons(port);
		memcpy(&destinataire.sin_addr, host->h_addr, host->h_length);
		for(int i=0; i<8; i++) destinataire.sin_zero[i] = 0;
 
		// Initialisation du socket
		sock = socket(AF_INET, SOCK_STREAM, 0);
		if(sock == -1){
			std::cerr << "Erreur à l'ouverture du socket\n" << std::endl;
			res = false;
		} else { 
 			//b= bind(sock, (SOCKADDR *)&destinataire, sizeof(destinataire));
		}
 
		connecte = connect(sock, (SOCKADDR*)&destinataire, sizeof(destinataire));
		if(connecte == -1){	
			std::cerr << "Echec de la connexion à " << nomDest << std::endl;
			res = false;
		} else {
			std::cout << "Connecté" << std::endl;
		}
	}
 
	return res;
}

Pour le client la connexion se fait dans une fonction d'initialisation, et l'on n'envoie ni ne reçoit rien pour le moment ...

Le serveur reste bloqué en attente de connexin alors que le client affiche connecté ...

17/07/2006, 14h36
Médinoc

listen() met seulement le socket dans un état d'écoute.
La fonction d'attente, c'est accept().
17/07/2006, 14h39
Ange44

Oui je sais ...

En fait il affiche bien le message "en attente de connexion" (oui je sais je mélange le C et le C++ mais je le fais à la va vite ce serveur c'est juste un petit truc pour tester mon client)

Et après il ne valide jamais le accept ... :( ... que le client se considère bien comme connecté ...
17/07/2006, 14h48
Médinoc

Ah, je n'avais pas vu le accept après ta boucle de listen() qui me tirait l'oeil.

D'ailleurs, renonces-y (aux deux boucles) : Car c'est le genre d'opération "qui a foiré, foirera" qui fera plonger ton programme dans une boucle infinie...
17/07/2006, 14h52
Ange44

Ok ...

Mais ça ne marche toujours pas ... il attend toujours une connexino alors que le client se considère comme connecté ...
17/07/2006, 14h52
Médinoc

Au fait : puisque tu es en winsock, utilise winsock jusqu'au bout: Tes sockets doivent être des SOCKET (un type non-signé) et non des int, et tu dois comparer à INVALID_SOCKET et non -1 (pour le retour de accept()).

Si ça trouve, tu as une erreur de signe, mais je n'y crois pas trop.

PS: Je crois que j'ai trouvé! longAddr n'est pas initialisé, alors que c'est une valeur in/out et non pas out...

PPS: C'est normal que le client se croie connecté. Le nombre maximum de clients qui se croient connectés est déterminé par le second paramètre de listen(), mais je crois que c'est toujours au moins 1 même si tu mets zéro...
17/07/2006, 14h57
cinemania

un listen(sock,1);
suffit, pas besoin de sortir une boucle avec des tests.
Ah oui le second paramètre ne doit jamais être nul ou négatif, mais toujours positif. Sinon ca n'a aucun sens. Par conséquent met 1 ou plus, on conseil générallement jusque 5, plus serait superflux. Dans ton cas, 1 me semble parfait.

Ensuite je suis d'accord travail, avec de l'arithmetique non signée, c'est souvent source de moins de probleme que l'arithmetique signée, et puis les constantes systemes sont là pour ca. Ca évite aussi les problemes, et générallement, sous linux par exemple, une acceptation avec une socket nulle renverra 0. Ca aussi c'est un cas d'erreur donc ...

En ce qui concerne l'acceptation... Je te suggere de casser ta boucle, ou du moins de rajouter une ligne supplémentaire pour indiquer à l'écran toute tentative de connexion infructueuse, donc tout résultat à -1. Cela serait suceptible d'être très intéressant et très instructif, car à priori je ne vois aucun probleme du coté du serveur.

Essaie donc d'ajouté la ligne :
if (acceptation == -1) printf("Echec d'acceptation. \n");
dans la boucle do {...} while (acceptation != -1)

En effet rassure toi, la fonction accept telle que tu l'a configurée est blocante, par conséquent, elle ne retourne un résultat qu'a l'occasion d'un évenement réseau sur la socket d'écoute.

je dirais même que dans ce cas la boucle est superflux.
17/07/2006, 15h06
Ange44

Merci c'était bien le champ longAddr qui n'était pas initialisé et qui bloquait l'accept() ...

Bon maintenant je vais essayer de déloquer les autres pb ... lol ... et je reviendrais demander conseille si je n'y arrive pas, mais là ça devrait aller...

Encore merci
17/07/2006, 15h10
Ange44

Euh en fait deux petites questions supplémentaries ...

1 ) Est-ce que la constante INVALID_SOCKET est à utiliser aussi pour vérifier si le bind, la lecture, ... sont bons ou alors il y a d'autres constantes système ?

2 ) Un socket est-il bidirectionnel ? (en gros le serveur et le client peuvent s'envoyer mutuellement des infos par le même socket ou non ?)
17/07/2006, 15h24
Médinoc
1. Sous Windows, on a tendance à utiliser SOCKET_ERROR au lieu de -1 pour pratiquement toutes les fonctions socket qui retournent autre chose qu'un socket: listen(), bind(), send(), recv(), etc, WSAStartup() étant l'exception la plus notable, puisqu'elle retourne directement le code d'erreur au lieu de -1.
  Tout cela est généralement décrit dans l'aide de chaque fonction sur la MSDN.
2. Oui.
17/07/2006, 15h33
Ange44

Les sockets sont donc bi-directionnels ...

Tant mieux, mais mon serveur ne veut pas envoyer de données ... après avoir réalisé la connexion, j'essaye de faire dans le serveur (pour envoyer au client) :

Code:

int n = send(sock, "coucou\r\n", 7, 0);

, et il me met n à SOCKET_ERROR ... donc il n'a pas envoyé ...
17/07/2006, 15h37
Médinoc

Il faut utiliser le socket retourné par accept() et non pas le socket d'écoute.
17/07/2006, 15h42
Ange44

Ok merci...

Bon ça a l'air de marcher, je vais faire le gros test maintenant ...

Encore meric pour tout