Probleme avec des socket !

**Ptimath44** · 30/10/2005, 18h03

Bonjour a tous,

Je redebute en C et c'est pas facile facile de recommencer par un pb de socket.
Si qq'1 a la moindre idée pourquoi mes 2 parties ne communiquent pas correctement, je suis preneur de toutes explications.
De même si vous avez un site cool qui explique l'utilisation de winsock.h, n'hésiter pas a faire signe.

Merci d'avance,

Mathieu

Partie 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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <winsock2.h>
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc,char *argv[])
{
char *prog;
WSADATA WSAData;
WSAStartup(MAKEWORD(2,0), &WSAData);
SOCKET sock;
SOCKADDR_IN sin;
char buffer[256];
char *msg;
char *host;
int boucl,erreur;
host = argv[1];
msg = argv[2];
sock = socket(AF_INET, SOCK_STREAM, 0);
if (sock==INVALID_SOCKET)
     printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d",WSAGetLastError());
sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
sin.sin_family				= AF_INET;
sin.sin_port				= htons(23);
erreur = connect(sock, (SOCKADDR *)&sin, sizeof(sin));
 if (erreur!=0)
         printf("\nDesole, je n'ai pas pu ouvrir la session TCP : %d %d",erreur,WSAGetLastError());
prog=argv[0];
 
if (argc != 3){
         printf ("Il faut 2 arguments pour executer ce programme");
         exit (1);
         }
 
//Envoyer la requete au serveur
printf("envoi de la requete serveur \n");
write (sock,msg,strlen (msg));
//lecture de la reponse du serveur
while ((boucl=read(sock,buffer,sizeof(buffer)))>0)
      write(1,buffer,boucl);
 
closesocket(sock);
WSACleanup();
 
exit;
}

Partie 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
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
#include <winsock2.h>
#include <stdio.h>
#include <stdlib.h>
 
void renvoi (int s);
 
int main()
{
   char *buffer[256]; 
    int nsd,toto;
WSADATA WSAData;
WSAStartup(MAKEWORD(2,0), &WSAData);
SOCKET sock;
SOCKET csock;
SOCKADDR_IN sin;
SOCKADDR_IN csin;
sock = socket(AF_INET, SOCK_STREAM, 0);
sin.sin_addr.s_addr			= INADDR_ANY;
sin.sin_family				= AF_INET;
sin.sin_port				= htons(23);
bind(sock, (SOCKADDR *)&sin, sizeof(sin));
listen(sock, 5);
    while(1)
    {
     int sinsize = sizeof(csin);
     if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
               {
                send(csock, "Bonjour Mathieu!\r\n", 14, 0);
                }
 
 
       renvoi (csock);
       close (csock);
    }
}
 
 
/////Fonction de renvoi
void renvoi (int s)
{
     char buff[256];
     char *fichier;
     char f[20];
     char *mesgptr;
     char mesg[250];
     int l, c;
     FILE *fp;
     char line [250]={0};
     char *lineptr ;
     // lire une requete
     if ((l=read(s,buff,sizeof(buff))) <=0) return;
     buff [1]='\0';//fin de chaine
     fichier=f;
     if (!strcmp(buff,"janvier")) fichier ="janvier";
          if (!strcmp(buff,"fevrier")) fichier ="fevrier";
               if (!strcmp(buff,"mars")) fichier ="mars";
                    if (!strcmp(buff,"avril")) fichier ="avril";
                         if (!strcmp(buff,"Mai")) fichier ="mai";
                              if (!strcmp(buff,"juin")) fichier ="juin";
                                   if (!strcmp(buff,"juillet")) fichier ="juillet";
                                        if (!strcmp(buff,"aout")) fichier ="aout";
                                             if (!strcmp(buff,"septembre")) fichier ="septembre";
if ((fp=fopen(fichier,"r"))==NULL)  printf("erreur lors de l'ouverture du fichier");
                                   while (fgets (line,250,fp)){
                                         write (s,line,strlen(line));
                                         }
 
           return ;                                             
 
}

**Médinoc** · 30/10/2005, 20h16

1) En quoi exactement ils "ne communiquent pas correctement" ?
Informations erronées? Attente sans fin? erreurs?

2) Le code de renvoi() est quelque peu inintelligible...

**David.Schris** · 30/10/2005, 21h09

Je suis preneur de toute explication concernant le problème et de quelle façon il se manifeste : il ne suffit pas de poster du code et d'attendre qu'on le teste et qu'on le débogue.

En attendant, essayez de faire plus simple en suivant un tutoriel.

**Skyounet** · 30/10/2005, 21h50

Envoyé par David.Schris

En attendant, essayez de faire plus simple en suivant un tutoriel.

Encore plus proche
http://c.developpez.com/WalrusSock/

L'indentation du serveur est a revoir.

**gentil_robot** · 30/10/2005, 21h51

Envoyé par Médinoc

2) Le code de renvoi() est quelque peu inintelligible...

Et je ne parle pas du test sur argc au milieu de la fonction

**David.Schris** · 30/10/2005, 22h50

Envoyé par gentil_robot

Et je ne parle pas du test sur argc au milieu de la fonction

Euh...si, tu en parles

Il me semble qu'il manque un WSACleanup(); dans le serveur, que le serveur ne se termine pas très correctement (pas de "return" ni de "exit()") et que l'appel à "exit()" dans le client souffre d'un manque de parenthèses (c'est une fonction attendant un int en paramètre). Par contre, le "return" à la fin de "renvoi()" n'est peut-être pas utile puisque, justement, elle ne renvoie rien. Plus les mélanges de "close()" et "closesocket()"...
Mais je n'ai pas tout lu : comme dit précédemment, j'attends d'abord des explications concernant le problème.

**Médinoc** · 30/10/2005, 23h25

t'inquiète pas pour le main du serveur, il n'est pas suppsé se terminer de toute façon (boucle infinie sans condition de sortie: ce qui importe pour l'instant c'est la comunication avec le client qui "déconne", sans plus de détails...

**Ptimath44** · 31/10/2005, 10h49

Merci pour toutes vos reactions vives !

Donc je vous explique ce qu'il en est :
Le serveur est en attente sur le 23.
la fonction renvoi doit renvoyer au client fonction du mois qu'il entre le contenu d'un petit fichier (janvier,fevrier,etc).
Coté client, on le lance avec l'IP serveur et le mois en deuxieme argument.
Actuellement le serveur tourne mais la communication avec le client ne semble pas opérationnelle.
De plus comme je vous l'ai dit, je redébute en programmation.
Le but de ce TP étant plus de faire de l'analyse de trame avec Ethereal ou autre sniffer.
Merci pour votre aide

++

**Emmanuel Delahaye** · 31/10/2005, 12h41

Envoyé par Ptimath44

Je redebute en C et c'est pas facile facile de recommencer par un pb de socket.

Est-il nécessaire d'écrire du code aussi mal ? N'est il pas possible d'écrire du code clair ?

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
 
/* client.c */
#include <winsock2.h>
#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
    WSADATA WSAData;
    WSAStartup (MAKEWORD(2,0), &WSAData);
 
    SOCKET sock = socket(AF_INET, SOCK_STREAM, 0);
 
    if (sock==INVALID_SOCKET)
    {
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d\n"
               ,WSAGetLastError());
    }
    else
    {
        SOCKADDR_IN sin;
 
        sin.sin_addr.s_addr = inet_addr("127.0.0.1");
        sin.sin_family      = AF_INET;
        sin.sin_port        = htons(23);
        {
            int erreur = connect(sock, (SOCKADDR *)&sin, sizeof(sin));
 
            if (erreur!=0)
            {
                printf("\nDesole, je n'ai pas pu ouvrir la session TCP : %d %d\n"
                       ,erreur
                       ,WSAGetLastError());
            }
            else
            {
 
                //Envoyer la requete au serveur
                printf("envoi de la requete serveur \n");
                {
                    char *msg = "requete au serveur\n";
                    send (sock,msg,strlen (msg),0);
                }
 
                //lecture de la reponse du serveur
                {
                    char buffer[256];
 
                    while (recv(sock,buffer,sizeof buffer,0)>0)
                    {
                        putchar(buffer[0]);
                    }
                }
            }
        }
        closesocket(sock);
    }
    WSACleanup();
    return 0;
}

et

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
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
 
/* server.c */
#include <winsock2.h>
#include <stdio.h>
#include <stdlib.h>
 
// Fonction de renvoi
void renvoi (SOCKET s)
{
    char buff[256];
    char *fichier;
    // lire une requete
    int l=recv(s,buff,sizeof buff-1,0);
 
    if (l >0)
    {
        buff [l]='\0';//fin de chaine
        fichier = buff;
 
        FILE *fp=fopen(fichier,"r");
 
        if (fp==NULL)
        {
            printf("erreur lors de l'ouverture du fichier\n");
        }
        else
        {
            char line [250];
            while (fgets (line,sizeof line,fp))
            {
                send (s,line,strlen(line),0);
            }
            fclose(fp);
        }
    }
}
 
int main (void)
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
 
    SOCKET sock;
    sock = socket(AF_INET, SOCK_STREAM, 0);
 
    if (sock==INVALID_SOCKET)
    {
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d\n"
               ,WSAGetLastError());
    }
    else
    {
        SOCKADDR_IN sin;
        sin.sin_addr.s_addr         = INADDR_ANY;
        sin.sin_family            = AF_INET;
        sin.sin_port            = htons(23);
        bind(sock, (SOCKADDR *)&sin, sizeof(sin));
        listen(sock, 5);
 
        while(1)
        {
            SOCKADDR_IN csin;
            int sinsize = sizeof(csin);
            SOCKET csock = accept(sock, (SOCKADDR *)&csin, &sinsize);
 
            if(csock != INVALID_SOCKET)
            {
                char *s_welcome = "Welcome on the Server\r\n";
 
                send(csock, s_welcome, strlen(s_welcome), 0);
                renvoi (csock);
                closesocket (csock);
            }
            else
            {
                break;
            }
        }
    }
    return 0;
}

**Ptimath44** · 31/10/2005, 15h26

Bien que j'ai ri un peu jaune a la lecture de vos post, je vous remercie tous.
Le petit client serveur se porte bien maintenant.

Prochaine étape les RPC mais en attendant il faut que je maitrise les socket basique et la ya deja du boulot.

Merci à tous
Ptimath

**David.Schris** · 31/10/2005, 19h08

Envoyé par Emmanuel Delahaye

Code :

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

1
2
3
4
5
6
7
8
9
10
 
/* server.c */
[...]
    if (sock==INVALID_SOCKET)
    {
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d\n"
               ,WSAGetLastError());
    }
    else
[...]

Mmmh...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/* server.c */
[...]
    if (sock==INVALID_SOCKET)
    {
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d\n"
               ,WSAGetLastError());
        WSACleanup();
        return 1;
    }
    else
[...]

Non ?

**Emmanuel Delahaye** · 31/10/2005, 19h11

Envoyé par David.Schris

Mmmh...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/* server.c */
[...]
    if (sock==INVALID_SOCKET)
    {
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d\n"
               ,WSAGetLastError());
        WSACleanup();
        return 1;
    }
    else
[...]

Non ?