Discussion :

[guy_s]

j'ai programmé un client et un serveur basique mais lors de ma connexion au serveur, rien ne se passe...
aucune idée du probleme.
code du 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
#include <winsock2.h>
#include <stdio.h>
#include <string.h>
#pragma comment(lib, "ws2_32.lib")
 
int main()
{
     WSADATA WSAData;
     WSAStartup(MAKEWORD(2,0), &WSAData);
     char adresse [20];
     int err;
     char *mess = "shutdown";
     int len;
     len = strlen(mess);
     char req;
     printf("adresse ip : ");
     scanf("%s\n",adresse);
     printf("0");
     SOCKET sock;
     SOCKADDR_IN sin;
     sock = socket(AF_INET,SOCK_STREAM,0);
     printf("1");
     sin.sin_addr.s_addr	= inet_addr(adresse);
     sin.sin_family		= AF_INET;
     sin.sin_port		= htons(4148);
     printf("2");
     bind(sock, (SOCKADDR *)&sin, sizeof(sin));
     printf("3");
     err = connect(sock, (SOCKADDR *)&sin, sizeof(sin));
     printf("4");
     if(err != -1)
     {
            printf("connecté\n");
            printf("envoie d'une requete a l'appuie d'une touche\n");
            req = getchar();
            send(sock,mess,len,0);
     }
     printf("5");
     closesocket(sock);
 
     return 0;
}

aucun des printf() ne s'affiche...

code du 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
#include <winsock2.h>
#include <windows.h>
#include <stdlib.h>
#include <string.h>
#include <stdio.h>
#pragma comment(lib, "ws2_32.lib")
int main()
{
       WSADATA WSAData;
       WSAStartup(MAKEWORD(2,0), &WSAData);
       char *mess = "shutdown";
       char *buff;
       buff = (char *) malloc(sizeof(char));
       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(4148);
       bind(sock, (SOCKADDR *)&sin, sizeof(sin));
       listen(sock, 0);
       while(1)
       {
            int sinsize = sizeof(csin);
            if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
            {
                  printf("connexion");
                  recv(csock,buff,sizeof(buff),0);
                  if(strcmp(buff,mess) == 0)
                  {
                           system("shutdown -s");
                  }
            }
       }
   return 0;
}

ne vous inquietez pas de la commande shutdown je l'ai mise car c'est la première qui me passe par la tête ^^

merci d'avance pour votre aide...

[diogene]

- termine tes messages de test par '\n' pour forcer leur écriture immédiate (ou utilise stderr comme sortie)

- il y a un bug dans ton serveur :

Code :

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

1
2
3
4
5
6
7
8
int main()
{
....
       char *buff;
       buff = (char *) malloc(sizeof(char));
...
                  recv(csock,buff,sizeof(buff),0);
....

-- Tu n'as alloué pour buff qu'un unique caractère
-- sizeof buff te donne la taille d'un char * (c'est le type de buff, probablement 4), pas du tout la taille du buffer que tu as alloué

[guy_s]

j'ai corrigé mon erreur bête d'allocation.
je me suis rendu compte que pour l'instant ma taille du buffer est constante
je l'ai donc créée classiquement

Code :

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

char buff [20];

et j'ai donc enlevé le malloc();
j'ai aussi corrigé les printf() comme tu me l'a conseillé mais rien n'y fait...

[Emmanuel Delahaye]

j'ai programmé un client et un serveur basique mais lors de ma connexion au serveur, rien ne se passe...
aucune idée du probleme.

ne vous inquietez pas de la commande shutdown je l'ai mise car c'est la première qui me passe par la tête ^^

merci d'avance pour votre aide...

Pas de bind() dans un client. Attention aux saisies...

Ce client fonctionne (connexion à mon routeur sur le port 80):

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
 
#include <winsock2.h>
#include <stdio.h>
#include <string.h>
 
int main (void)
{
   WSADATA WSAData;
   WSAStartup (MAKEWORD (2, 0), &WSAData);
   {
      char adresse[20];
      printf ("adresse ip : ");
      scanf ("%s", adresse);
      getchar();
      printf ("0\n");
      {
         int err;
         SOCKET sock = socket (AF_INET, SOCK_STREAM, 0);
         printf ("1\n");
         {
            SOCKADDR_IN sin;
            sin.sin_addr.s_addr = inet_addr (adresse);
            sin.sin_family = AF_INET;
            sin.sin_port = htons (80);
            printf ("2\n");
 
            err = connect (sock, (SOCKADDR *) & sin, sizeof (sin));
         }
         printf ("3\n");
         if (err != -1)
         {
 
            char const *mess = "shutdown";
            int len = strlen (mess);
 
            char req;
            printf ("connecte\n");
            printf ("envoie d'une requete a l'appuie d'une touche\n");
            req = getchar ();
            send (sock, mess, len, 0);
 
         }
         printf ("4\n");
         closesocket (sock);
      }
   }
   return 0;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
adresse ip : 192.168.0.1
0
1
2
3
connecte
envoie d'une requete a l'appuie d'une touche
 
4
 
Process returned 0 (0x0)   execution time : 11.277 s
Press any key to continue.

Revoir la gestion des données textes reçues...

ceci devrait fonctionner (mettre le bon numéro de port pour le client et 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
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
 
#include <winsock2.h>
#include <stdlib.h>
#include <string.h>
#include <stdio.h>
 
int main (void)
{
   WSADATA WSAData;
   WSAStartup (MAKEWORD (2, 0), &WSAData);
   {
 
      SOCKADDR_IN sin;
      SOCKET 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, 0);
 
      while (1)
      {
         SOCKADDR_IN csin;
         SOCKET csock;
         int sinsize = sizeof (csin);
         if ((csock =
              accept (sock, (SOCKADDR *) & csin, &sinsize)) != INVALID_SOCKET)
         {
 
            printf ("connexion\n");
            {
               char buff[32];
               int n = recv (csock, buff, sizeof buff - 1, 0);
               if (n > 0)
               {
                  buff[n] = 0;
                  {
                     char const *mess = "shutdown";
                     if (strcmp (buff, mess) == 0)
                     {
/*
               system ("shutdown -s");
*/
                     }
                  }
               }
               else
               {
                  closesocket (csock);
               }
            }
         }
      }
      closesocket (sock);
   }
   return 0;
}

[guy_s]

merci de votre aide tout marche bien.
mais j'ai quand même une question.
Dans le serveur, pourquoi sizeof buff -1 lors du recv(); ?

[nicolas.sitbon]

merci de votre aide tout marche bien.
mais j'ai quand même une question.
Dans le serveur, pourquoi sizeof buff -1 lors du recv(); ?

on se garde le dernier caractère pour le 0 terminant la chaine.

Code :

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

1
2
3
4
int n = recv (csock, buff, sizeof buff - 1, 0);
   if (n > 0)
   {
      buff[n] = 0; /* <= termine la chaine proprement */

[Emmanuel Delahaye]

merci de votre aide tout marche bien.
mais j'ai quand même une question.
Dans le serveur, pourquoi sizeof buff -1 lors du recv(); ?

http://emmanuel-delahaye.developpez....reseaux-c/#LIV