Discussion :

[theawe]

Bonjour,

D'où pourrait provenir le fait qu'un recv, sous windows, est systématiquement égal à -1, alors que le send qui précède est correct ?

la valeur -1 ne devrait-elle pas, théoriquement, être renvoyée sous linux seulement ?

Je ne vois pas le problème dans mon appel à cette fonction : recv est appelée sur le serveur et j'utilise bien le socket du client pour ceci.

Auriez-vous une idée de la source du problème ?

[Médinoc]

la valeur -1 ne devrait-elle pas, théoriquement, être renvoyée sous linux seulement ?

Euh... Pourquoi?

Pour le reste de ton problème, impossible de répondre sans plus d'infos.

[theawe]

Euh... Pourquoi?

J'avais lu sur un tutoriel que la valeur de retour, en cas d'erreur, était SOCKET_ERROR sous windows ou -1 sous linux. Cependant, SOCKET_ERROR doit sans doute correspondre à une valeur numérique, donc peut-être -1 du coup...

Pour le reste de ton problème, impossible de répondre sans plus d'infos.

L'erreur générée est l'erreur 183 (identifiée avec WSAGetLastError()). Erreur 183 : "Cannot open a file when that file already exists." Je n'ouvre pourtant le fichier qu'après l'appel de recv. Je ne comprends pas vraiment pour cette fois-ci je ne parviens pas à recevoir mon message.

Le client appel send pour envoyer son message. L'appel de send ne génère pas d'erreur : les paramètres ont été vérifiés et sont corrects.

Le serveur appel recv pour récupérer ce message mais une erreur est générée.

Toutes les déclarations concernant les sockets, les SOCKADDR_IN, WSADATA et WSAStartup sont corrects. Le serveur reçoit correctement l'appel de connexion du client et l'accepte sans erreur.

Je n'ai pas plus d'informations à donner.

[Médinoc]

Peux-tu poster ton code, au moins depuis l'appel à send() jusqu'à l'appel à WSAGetLastError(), en passant par l'appel à recv()?

[Flow_75]

sous windows, SOCKET_ERROR est une macro qui correspond à -1

[theawe]

Côté client :

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
 
if (connect(socketClient, (SOCKADDR *)&sin, sizeof(sin)) == 0)//Si connexion
{
	CString message = m_strId + m_strPass;
 
	m_bufferSend = new char[50];
	std::string strMessage = CT2A(message);
	m_bufferSend = strMessage.c_str();
	if(send(socketClient, m_bufferSend, message.GetLength(), 0) != SOCKET_ERROR)
	{
		m_bufferRecv = new char[3];
		if(recv(socketClient, m_bufferRecv, sizeof(m_bufferRecv), 0) != SOCKET_ERROR)
		{
			CString messageRecv(m_bufferRecv, sizeof(m_bufferRecv));
			if(messageRecv == "yes")
			{
				bConnected == true;
				AfxMessageBox("La connexion est établie.");
			}
		}
	}
}

Côté serveur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
if((socketClient == accept(socketServeur, (SOCKADDR *)&sinClient, &sinsize)) != INVALID_SOCKET)
{
 
	int i = 0;
	i = recv(socketClient, m_bufferRecv, sizeof(m_bufferRecv), 0);
 
	FILE * pFile = fopen("test_Recv.txt", "a");
	fprintf(pFile, "Recv = %d\n", i);
	fprintf(pFile, "recv failed: %d\n", WSAGetLastError());
	fclose(pFile);
 
/* traitements */
}

Edit : j'ai fait un appel à WSAGetLastError() juste après l'appel de recv. L'erreur est en fait 10038 : Ici

C'est la première fois que je rencontre cette erreur et pourtant je n'ai ni changé de PC, ni changé d'adresse IP.

[Médinoc]

Attention, ton utilisation de sizeof est incorrecte.
De plus, tu devrais appeler WSAGetLastError() avant le fopen(), car fopen() repose sur l'API Win32 et peut modifier le LastError (sous Win32, il n'y a pas de LastError séparé entre WSA et le reste).

[theawe]

Je viens de tester la communication en utilisant une autre application Client-Serveur faite la semaine dernière et je ne rencontre aucun problème.

[theawe]

Attention, ton utilisation de sizeof est incorrecte.

Pourquoi cela ?

voici ce qui dit msdn à propos de mon erreur :
Socket operation on nonsocket.

An operation was attempted on something that is not a socket. Either the socket handle parameter did not reference a valid socket, or for select, a member of an fd_set was not valid.

Pourtant je passe bien dans ma condition de vérification de validité du socket...

[Médinoc]

Pourquoi cela ?

sizeof sur un pointeur donne la taille du pointeur, pas du tableau pointé...

[theawe]

sizeof sur un pointeur donne la taille du pointeur, pas du tableau pointé...

Je déclare mon buffer de la manière suivante :

Code :

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

1
2
 
char m_bufferRecv[2048];

Que je mette 2048 ou la taille du message à recevoir directement en paramètre à la place de sizeof(m_bufferRecv), le problème reste le même.

[Médinoc]

Code client :

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

1
2
		m_bufferRecv = new char[3];
		if(recv(socketClient, m_bufferRecv, sizeof(m_bufferRecv), 0) != SOCKET_ERROR)

D'accord, c'est le serveur qui foire, mais tu as quand même une erreur sur le client...

Edit: J'ai trouvé l'erreur! Il y a un '=' de trop dans la ligne du accept()...

[theawe]

Code client :

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

1
2
		m_bufferRecv = new char[3];
		if(recv(socketClient, m_bufferRecv, sizeof(m_bufferRecv), 0) != SOCKET_ERROR)

D'accord, c'est le serveur qui foire, mais tu as quand même une erreur sur le client...

Je vais modifier ça tout de suite. Je pensais que tu parlais du sizeof(m_bufferRecv) du côté serveur...

[CODE=client]
Edit: J'ai trouvé l'erreur! Il y a un '=' de trop dans la ligne du accept()...

Ah ! Super, merci...