Discussion :

[lcardao]

Bonjour a tous,

J'ai beau vérifier de fond en comble mais je ne trouve pas du tout ce qui provoque cette erreur de bind ....
Pour le contexte, je dois faire un réseau en multi thread en C++ sur Windows et la fonction si dessous me sert à créer une nouvelle socket d'écoute sur le réseau local.
J'utilise les lib <windows.h>, <thread>, <winsock2.h>, <ws2tcpip.h>, <fstream>, <string> , <vector>, <limits>, <iostream>, <stdlib.h>, <stdio.h> (je ne sais pas si c'est utile mais ne on sait jamais).

Voici mon code :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
SOCKET Init_Serveur() {
	SOCKET ListenSocket = INVALID_SOCKET;
	int iResult;
	struct addrinfo hints, *result = NULL;
 
	ZeroMemory(&hints, sizeof(hints));
	hints.ai_family = AF_INET;  // Address Format Internet
	hints.ai_socktype = SOCK_STREAM;  //  Provides sequenced, reliable, two-way, connection-based byte streams
	hints.ai_protocol = IPPROTO_TCP;  // Indicates that we will use TCP
	hints.ai_flags = AI_PASSIVE;  // Indicates that the socket will be passive => suitable for bind()
	if (iResult = getaddrinfo(NULL, DEFAULT_PORT, &hints, &result) != GOOD_EXEC) {
		std::cout << "Getaddrinfo failed with error: " << iResult << std::endl;
	}
	else {
		if (ListenSocket = socket(result->ai_family, result->ai_socktype,
			result->ai_protocol) == INVALID_SOCKET) {
			std::cout << "Socket failed with error: " << WSAGetLastError() << std::endl;
			ListenSocket = INVALID_SOCKET;
		}
		else {
			if (iResult = bind(ListenSocket, result->ai_addr,
				(int)result->ai_addrlen) == SOCKET_ERROR) {
				std::cout << "Bind failed with error: " << WSAGetLastError() << std::endl;
				closesocket(ListenSocket);
				ListenSocket = INVALID_SOCKET;
			}
			else {
				if (listen(ListenSocket, SOMAXCONN) == SOCKET_ERROR) {
					std::cout << "Listen failed with error: " << WSAGetLastError() << std::endl;
					closesocket(ListenSocket);
					ListenSocket = INVALID_SOCKET;
				}
			}
		}
	}		
	freeaddrinfo(result);
 
	return ListenSocket;
}

Merci d'avance de votre aide.

[Bousk]

10038 signifie que le paramètre n'est pas un socket valide.
Le code est peu lisible. Toute la magouille avec getaddrinfo est superflue.
Tes if sont étranges, même si j'apprécie cette syntaxe pour des pointeurs, faire une assignation puis un check de cette façon réduit la lisibilité, et avoir les if sur 1 ou 2 lignes n'aide pas.
Tous ces if en cascades n'ont également pour seul résultat que de diminuer la lisibilité.

Ouvrir un socket serveur TCP se fait en quelques lignes très simples

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
SOCKET server = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
	if (server == INVALID_SOCKET)
	{
		std::cout << "Erreur initialisation socket : " << WSAGetLastError();
		return -2;
	}
 
	const unsigned short port = 9999;
	sockaddr_in addr;
	addr.sin_addr.s_addr = INADDR_ANY;
	addr.sin_port = htons(port);
	addr.sin_family = AF_INET;
 
	int res = bind(server, (sockaddr*)&addr, sizeof(addr));
	if (res != 0)
	{
		std::cout << "Erreur bind : " << WSAGetLastError();
		return -3;
	}
 
	res = listen(server, SOMAXCONN);
	if (res != 0)
	{
		std::cout << "Erreur listen : " << WSAGetLastError();
		return -4;
	}

Sans oublier d'initialiser WinSock au préalable bien sûr.

Pour le reste : https://bousk.developpez.com/cours/r...iers-pas/#LIII