Probleme avec select

**sorry60** · 16/12/2005, 12h04

Salut,

J'ai un probleme avec la fonction select

Ma fonction est "en attente" de reception (mais non bloquante).
Voici son 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
/* Fonction en attente de reception d'un message sous la forme login : message */
void receive(gpointer donnees)
{
	data* pData = donnees;
	if ( pData->verifConnect == 1 )
	{
		struct timeval tv;
		int retselect;
 
		printf("Deb fonction\n");		
		FD_ZERO(&(pData->monFds));
		FD_SET(pData->maSocket,&(pData->monFds));
		memset(&tv, 0, sizeof(tv));
      retselect = select((pData->maSocket) + 1,&(pData->monFds),NULL,NULL,&tv);
      printf("reselect = %d\n",retselect);
	}
}

Le code n'est bien evidemment qu'en phase d'approche

voici le probleme : dèsque j'envoie un message, retselect passe à 1 (ok jusqu'ici tout va bien) mais il le reste indefinement pour chaque appel de la fonction

Pourtant le fds de lecture est bien réinitialisé avant chaque appel à select :

Code :

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

1
2
FD_ZERO(&(pData->monFds));
FD_SET(pData->maSocket,&(pData->monFds));

Voici la structure data :

Code :

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

1
2
3
4
5
6
7
8
typedef struct
	{
		GtkWidget *pFenetre;			/* Fenetre principale */
		/* ... */
		SOCKET maSocket;
		fd_set monFds;
		int verifConnect;					/* 1 ==> connecté au serveur, sinon 0 */
	}data;

Je n'arrive pas à comprendre pourquoi le fds n'est pas correctement réinitialisé, ou alors pourquoi select renvoie toujours 1, alors que je n'envoie qu'UN seul message

Merci pour votre aide
Sorry

**fearyourself** · 16/12/2005, 12h44

Code :

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

retselect = select((pData->maSocket) + 1,&(pData->monFds),NULL,NULL,&tv);

Select prévient qu'une donnée est prête à être lue mais si tu ne vides pas le flux en lisant select rendra tourjous 1 disant "le flux a encore des données"...

**sorry60** · 16/12/2005, 13h16

Envoyé par fearyourself

Code :

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

retselect = select((pData->maSocket) + 1,&(pData->monFds),NULL,NULL,&tv);

Select prévient qu'une donnée est prête à être lue mais si tu ne vides pas le flux en lisant select rendra tourjous 1 disant "le flux a encore des données"...

Ok

**sorry60** · 16/12/2005, 13h35

J'ai complété ma fonction :

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
void receive(gpointer donnees)
{
	data* pData = donnees;
	if ( pData->verifConnect == 1 )
	{
		fd_set rfds;
		struct timeval tv;
		int retselect;
		char buffer[127];
 
		printf("Deb fonction\n");		
		FD_ZERO(&rfds);
		FD_SET(pData->maSocket,&rfds);
		memset(&tv, 0, sizeof(tv));
      retselect = select((pData->maSocket) + 1,&rfds,NULL,NULL,&tv);
      printf("reselect = %d\n",retselect);
      if ( retselect > 0 )
      {
      	recv(pData->maSocket,buffer,127,0);
      	puts(buffer);
      }
	}
}

Mais le problème n'est pas réglé :

au premier passage dans la fonction apres l'envoie du message, retselect passe bien à 1 et le message est affiché

mais pour tous les passages suivants, retselect reste à 1 et n'importe quoi est reçu et affiché

Voici le bout de code qui envoit le message, si jamais le probleme vient de là...

Code :

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

1
2
3
char erreur404[127] = "Desole vous n'etes pas dans la base de donnees...\0";
int oct_env;
oct_env = send(newClient,erreur404,127,0);

**fearyourself** · 16/12/2005, 13h53

Je ne mettrais pas ceci:

Code :

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

1
2
 
char erreur404[127] = "Desole vous n'etes pas dans la base de donnees...\0";

Mais:

Code :

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

1
2
 
char erreur404[127] = "Desole vous n'etes pas dans la base de donnees...";

Car le '\0' se met automatiquement

Je ne mettrais pas ceci:

Code :

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

oct_env = send(newClient,erreur404,127,0);

Mais:

Code :

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

oct_env = send(newClient,erreur404,strlen(erreur404),0);

Jc

**sorry60** · 16/12/2005, 14h29

ok j'ai corrigé, merci pour la correction.
Malheureusement ça ne règle pas le problème

recv continue de trop recevoir

**fearyourself** · 16/12/2005, 14h42

Est-il possible que tu fermes la connection du côté serveur ou que la connection se ferme entre temps? Je crois me souvenir d'un problème si la socket se fermait mais je peux me tromper...

**sorry60** · 16/12/2005, 15h21

Envoyé par fearyourself

Est-il possible que tu fermes la connection du côté serveur ou que la connection se ferme entre temps? Je crois me souvenir d'un problème si la socket se fermait mais je peux me tromper...

Oui en effet :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/* Client inconnu ! */
char erreur404[127] = "Desole vous n'etes pas dans la base de donnees...";
int oct_env;
/* On envoie un message d'erreur au client inconnu */
oct_env = send(newClient,erreur404,strlen(erreur404),0);
if ( oct_env == -1 )
{
   perror("send() 404error");
}
/* On ferme la socket */
closesocket(newClient);

Mais mon serveur, doit ici refuser la connection du client (car pas connu) donc je ferme la socket.

Edit : un shutdown avant le closesocket serait peut etre mieux, non ?

**fearyourself** · 16/12/2005, 15h27

Le client n'a qu'à vérifier que le serveur à fermer la socket et donc mettre ta variable verifConnect à 0...

recv doit rendre -1 s'il y a un problème, je suppose que cela inclut que le socket soit fermé de l'autre côté... Il faudrait vérifier...

**sorry60** · 16/12/2005, 15h30

Envoyé par fearyourself

recv doit rendre -1 s'il y a un problème, je suppose que cela inclut que le socket soit fermé de l'autre côté... Il faudrait vérifier...

Exact, je suis idiot de pas avoir testé le retour de recv avant d'afficher...

**Médinoc** · 16/12/2005, 15h37

lorsque le socket est fermé proprement, recv() ne retourne pas -1 mais zéro...

**fearyourself** · 16/12/2005, 15h39

lorsque le socket est fermé proprement, recv() ne retourne pas -1 mais zéro...

Exact, de la page man:

These calls return the number of bytes received, or -1 if an error occurred. The return value will be 0 when the peer has performed an orderly shutdown.

Jc

**sorry60** · 16/12/2005, 15h51

Ok, donc j'ai corrigé ma fonction, voici la derniere version :

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
41
void receive(gpointer donnees)
{
	data* pData = donnees;
	printf("VerifConnect = %d\n",pData->verifConnect);
	if ( pData->verifConnect == 1 )
	{
		fd_set rfds;
		struct timeval tv;
		int retselect;
		char buffer[127];
 
		printf("Deb fonction\n");
		FD_ZERO(&rfds);
		FD_SET(pData->maSocket,&rfds);
		memset(&tv, 0, sizeof(tv));
      retselect = select((pData->maSocket) + 1,&rfds,NULL,NULL,&tv);
      printf("retselect : %d\n",retselect);
      if ( retselect > 0 )
      {
      	int recsize = recv(pData->maSocket,buffer,127,0);
      	if (  recsize > 0)
      	{
      		printf("%d octets reçus : %s\n",recsize,buffer);
      	}
      	else
      	{
      		if ( recsize == 0 )
      		{
		  			printf("Fermeture socket...\n");
      			shutdown(pData->maSocket,2);
      			closesocket(pData->maSocket);
      			pData->verifConnect = 0;
      		}
      		else
      		{
      			perror("recv()");
      		}
      	}
      }
	}
}

Donc je me connecte au serveur, et lui envoie un login invalide.
Le message d'erreur est reçu, affiché mais ça deborde :

49 octets reçus : Desole vous n'etes pas dans la base de donnees...\uffff \uffff\uffff_\uffffb

De plus, ensuite ma fonction n'est plus appellée une fois que la socket est fermée, puisque seul :

VerifConnect = 1

est écrit en dessous..des qu'il repasse à 0 plus rien n'est affiché donc le programme n'appelle plus la fonction

Je ne comprend plus rien

**fearyourself** · 16/12/2005, 15h57

Le '\0' n'est pas envoyé et donc pas reçu donc à l'affichage ça déborde...

C'est ma faute, je t'ai dit de mettre

Code :

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

1
2
 
send(erreur404,strlen(erreur404));

Or il faudrait mettre

Code :

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

1
2
 
send(erreur404,strlen(erreur404)+1); //pour prendre en compte le '\0'

Jc

EDITION: Par contre fais attention, si mes souvenirs sont bons, TCP permet de découper les trames (ségmentation des données) donc il se peut qu'un envoi par send se retrouve découper... Il faudrait donc plusieurs recv pour tout récupérer...

Une solution est de faire un select pour voir s'il y a quelque chose dans la socket et dès qu'il y a quelque chose, faire une boucle do..while(select..) tant qu'il ya quelque chose dans la socket

**sorry60** · 16/12/2005, 16h24

Ok merci en effet le probleme de debordement est maintentant corrigé.
Par contre, une fois la socket fermée, recv reçoit bien 0, donc je ferme la socket du client et met verifConnect à 0. Mais à partir de ce moment la fonction receive n'est plus du tout appellée

sauf si je rajoute à la fin de cette derniere un printf

**fearyourself** · 16/12/2005, 16h36

Tu as enlevé cette ligne aussi?

printf("VerifConnect = %d\n",pData->verifConnect);

Tu veux dire que si tu rajoutes un printf, tout marche mais sinon même le premier printf ne s'affiche plus?

Jc

**sorry60** · 16/12/2005, 16h45

Envoyé par fearyourself

Tu as enlevé cette ligne aussi?

printf("VerifConnect = %d\n",pData->verifConnect);

Tu veux dire que si tu rajoutes un printf, tout marche mais sinon même le premier printf ne s'affiche plus?

Jc

Non je laisse le 1er printf, comme ça je vois si je rentre ou non dans la fonction.
Quand je rajoute un printf à la fin j'y rentre tout le temps sans probleme, quand je le retire j'y rentre plus.
C'est peut etre le timer Gtk qui fait des siennes...

Code :

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

g_timeout_add(100,(GSourceFunc)receive,&myData);

**Médinoc** · 16/12/2005, 17h13

C'est réglé: cet ahuri avait mal déclaré sa callback : Elle est supposée retourner un gboolean qui retire le timer si FALSE...

**sorry60** · 16/12/2005, 17h16

Envoyé par Médinoc

C'est réglé: cet ahuri avait mal déclaré sa callback

Pas gentil.

**Médinoc** · 16/12/2005, 17h18

Pas gentil peut-être, mais ça t'avait été dit presque dès le début, et tu n'avais pas corrigé!
Et en plus, j'avais un autre mot qu'ahuri aux doigts, je me suis autocensuré par politesse...

Edit: 'fait suer, pas moyen de retrouver l'image du clavier abrutyuiop...

Probleme avec select

Réseau C

Discussions similaires

Partager

Partager