Discussion :

[Sykaa]

Bonjours tous le monde.
Ça semble simple, ça doit probablement l'Être, mais là je seche.
Je m'entriane tranquillement à faire dialoguer un client et un serveur tout bêtes, avec de simple chaines de caractère mais au moment de fiare un read() par le client, apres un write du serveur, le programme client plante et retourne un une segmentation fault.
Le pire c'est que ça marche tres bien dans un sens : si mon client ne fait qu'ecrire sans lire, le serveur lit sans probleme et tous se déroule correctement. C'est lorsque le serveur ecrit quelque chose sur le socket que là le client plante en voulant lire.

Le 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
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 int fd_client = accept (fd, &client,(socklen_t*)&client_len);
        pid_t p_client = fork ();
 
 
        if (p_client == 0)
  	{ 
	  char buf[301];
	  int n;
	  close( fd);
 
	  while(strcmp(buf,"END") != 0){
 
 
    	    if((n = read (fd_client, buf, 300)) >= 0)
    	      {
 
    		buf[n] = '\0';
    		cout<<buf<<"\n";
    	      }
 
 
	    int b = write (fd_client,"OK", 7);
 
	  }
 
	  close (fd_client);
  	  exit (0);
 
  	}

et le 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
char buf[301]="";
  int n;
  char* s;
  while(strcmp(s,"q") !=0){
 
    printf ("client connecté à %s (%d.%d.%d.%d) port=%d\n",
	    server_addr,
	    (ntohl (server.sin_addr.s_addr) >> 24) & 0xff,
	    (ntohl (server.sin_addr.s_addr) >> 16) & 0xff,
	    (ntohl (server.sin_addr.s_addr) >> 8) & 0xff,
	    (ntohl (server.sin_addr.s_addr) >> 0) & 0xff,
	    (ntohs (server.sin_port)));
    scanf("%s",s);
    write (fd, s, 7);
     if ((n = read (fd, buf, 300)) <= 0){ //voila le read en question qui ne veut pas se faire
       buf[n] = '\0';                            //si j'enleve tous ça l'affaire fonctionne
       cout << buf<<"\n";
     }
 
  }

Merci à ceux qui ont pris le temps de lire et de reflechir un peu.

[nicolas.sitbon]

Ton code est du C++ et tu es dans un forum C...
J'ai particulièrement apprécié le :

Le serveur

Code :

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

...

et le serveur

Code :

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

...

Enfin, j'ai de gros doute la dessus:

Code :

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

1
2
3
4
5
 
char* s;
while(strcmp(s,"q") !=0)
scanf("%s",s);
write (fd, s, 7);

tu crois faire quoi là?

[Sykaa]

Effectivement le coup du serveur serveur... Enfin j'ai corrigé.
Sinon concernant le coté C++ du code mis à part les "cout", je crois que tous le reste est du C? ou je me trompe?

Pour ce qui est de ta remarque sur

Code :

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

1
2
3
4
5
 
char* s;
while(strcmp(s,"q") !=0)
scanf("%s",s);
write (fd, s, 7);

c'est une boucle pour lire ce que j'écris au clavier, qui s'arrette si je tape q, et qui envoie ce que j'ai tapé au server. (fd étant le déscripteur de fichier du socket connecté au serveur).
J'aurai certe dû initialisé ma variable (chose faite), mais cette boucle j'ai l'impression qu'elle marche si j'empeche le client de "lire" le socket. Je peux ecrire tout ce que je veux et c'est envoyé au serveur qui me l'affiche (un peu nul mais faut bien se faire la main...), jusqu'à que j'envoye un END (qui ferme la connexion du coté serveur) ou un q, qui ferme la connexion coté client.

En tout cas merci d'avoir jeté un coup d'oeil.

Je précise que j'ai passé tout ça sous gdb, et que le Segmentation Fault apparait vraiment au moment d'executer le "read()", je n'arrive même pas a entre dans la fonction à l'aide du step.

[nicolas.sitbon]

Je repose ma question, que crois tu faire en faisant ça:

Code :

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

1
2
char* s;
scanf("%s",s);

rien ne te choque?

[Sykaa]

Hum, je reflechi...
le %s? qui devrait peut-être être un %c? non.
Scanf a besoin d'un tableau de taille définie, et donc il me faudrait déclarer s plutot du genre :

Code :

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

char s[20];

je me rapproche?

[Emmanuel Delahaye]

Hum, je reflechi...
le %s? qui devrait peut-être être un %c? non.
Scanf a besoin d'un tableau de taille définie, et donc il me faudrait definir s plutot du genre :

Code :

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

char s[20];

je me rapproche?

C'est déjà mieux, oui...

[Emmanuel Delahaye]

Sinon concernant le coté C++ du code mis à part les "cout", je crois que tous le reste est du C? ou je me trompe?

C'est du C uniquement si c'est compilé avec un compilateur C...

Pour ce qui est de ta remarque sur

Code :

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

1
2
3
4
5
 
char* s;
while(strcmp(s,"q") !=0)
scanf("%s",s);
write (fd, s, 7);

c'est une boucle pour lire ce que j'écris au clavier, qui s'arrette si je tape q, et qui envoie ce que j'ai tapé au server. (fd étant le déscripteur de fichier du socket connecté au serveur).
J'aurai certe dû initialisé ma variable (chose faite), mais cette boucle j'ai

Gné ? On peut voir le vrai code qui pose problème ?

[Sykaa]

C'est du C uniquement si c'est compilé avec un compilateur C...

Gné ? On peut voir le vrai code qui pose problème ?

Le code qui "posait" problème, est dans mon premier post.
"posait" car le seg fault n'est plus... Merci. Le comportement n'est pas encore tout a fait celui que j'attendais, mais au moins je ne bloque plus.
Il va par contre falloir que j'essaye de comprendre pour quelles raisons celà posait problème au read(), qui n'avait à premiere vu rien à voir avec la vilaine chaine de caractères non constante que j'avait déclarer..
Encore merci, et désolé pour la pollution C++...

[Emmanuel Delahaye]

Il va par contre falloir que j'essaye de comprendre pour quelles raisons celà posait problème au read(), qui n'avait à premiere vu rien à voir avec la vilaine chaine de caractères non constante que j'avait déclarer..

Ne perd pas ton temps à expliquer un comportement indéfini. Tout peut arriver. Point. Passe plutôt du temps à comprendre comment écrire du code sans comportements indéfinis...