[Débutant][Sockets] Server en C++, client en Java

**mavina** · 02/10/2009, 08h03

Salut,

Je m'ammuse en ce moment à faire un petit daemon pour les serveurs de ma boite, et je voudrais le faire en C++ histoire d'avoir un exécutable et pas être obligé d'avoir java qui tourne... Enfin bref.

J'ai suivi plusieurs tutoriels et pioché à droite à gauche pour faire un serveur en C++ qui écoute sur une socket sur le port 15000.
Il attend bien la connexion, "logs à l'appui" (en supposant que ca plante si ca fonctionne pas

).

Bon, sans plus attendre, le code de la bête :

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
 
#include <cstdlib>
#include <iostream>
#include <winsock2.h>
#pragma comment(lib, "ws2_32.lib")
 
using namespace std;
 
int main(int argc, char *argv[])
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
 
    SOCKET sock;
    SOCKADDR_IN sin;
    sin.sin_addr.s_addr	= inet_addr("127.0.0.1");
    sin.sin_family		= AF_INET;
    sin.sin_port		= htons(15000);
    sock = socket(AF_INET,SOCK_STREAM,0);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
 
    listen(sock, 0);
    int val = 0;
 
    int sizeSin = sizeof(sin);
 
    cout << "created socket on port 15000" << endl;
    cout << "now listening" << endl;
 
    while(1)
    {
        val = accept(sock, (SOCKADDR *)&sin, &sizeSin);
        if(val != INVALID_SOCKET)
        {
               char* buffer=new char[1024];
               int retour = 0;
               retour = recv(sock,buffer,sizeof(buffer),0 );
               if (retour == SOCKET_ERROR) 
               {
                  cout << "erreur lors de la lecture" << endl << "errno : " << WSAGetLastError() << endl;
                  continue;
               }
               else
               {
                   if(strcmp(buffer,"SHUTDOWN")==0)
                   {
                       cout << "SHUTDOWN RECEIVED"<<endl;
                   }
               }
        }
    }
 
 
    cout << "ciyciy" << endl;
    cout << "test" << endl;
    system("PAUSE");
    //return EXIT_SUCCESS;
}

et mon client en java qui se connecte, envoie SHUTDOWN et se déconnecte :

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
 
import java.net.*;
 
public class TestDaemon
{
	TestDaemon()
	{
	}
 
	public static void main(String[] args)
	{
		try
		{
			Socket s=new Socket("localhost",15000);
			s.getOutputStream().write("SHUTDOWN".getBytes(),0,8);
			s.getOutputStream().flush();
			s.close();
		}
		catch(Exception e)
		{
			e.printStackTrace();
		}
 
	}
}

Le client ne me renvoie aucune erreur.

Voici le log du serveur :

created socket on port 15000
now listening
erreur lors de la lecture
errno : 10057

J'ai un peu cherché sur le net à propos de l'erreur 10057 mais j'ai soit rien compris soit rien trouvé d'interressant. Visiblement la socket est pas connectée et donc a refusé la connexion (

) d'où mon incompréhension.

Alors ma question est : D'où que ca cloche ? C'est grave docteur ?

Autre chose, il semblerai que, puisque j'ai donné comme inet_address 127.0.0.1 à ma socket, elle n'accepte que les connexions de moi sur moi même, et pas de l'extérieur. Si je ne mets pas de inet_address, alors le socket n'accepte aucune connexion. Comment faire pour qu'il accepte la connexion de n'importe quelle ip ?

Merci d'avance

F.

**mavina** · 02/10/2009, 09h37

Bonjour,

J'ai trouvé pourquoi la socket n'était pas connectée, il faut écouter avec la socket que accept nous renvoie.

Maintenant j'ai un nouveau probleme :

J'en voie "SHUTDOWN" et je ne recois que "SHUT". Or, si j'envoie "SHUTDOWN coucou coucou coucou" je ne recois toujours que "SHUT".
Pourquoi ?

Merci d'avance.

Fred.

**ram-0000** · 02/10/2009, 09h48

Dans ton code :

Code :

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

1
2
char* buffer=new char[1024];
retour = recv(sock,buffer,sizeof(buffer),0 );

sizeof(buffer) vaut 4, c'est un pointeur d'où les 4 caractères que tu recois ("SHUT")

Il faudrait faire

Code :

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

retour = recv(sock,buffer,1024,0 );

et même mieux avoir une constante pour l'allocation et la réutiliser pour la lecture

**mavina** · 02/10/2009, 09h52

Salut,

merci de ta réponse rapide.

Je viens de faire ca apres m'être souvenu de mes vieux cours de C++ (qui commencent à dater mine de rien) que la taille d'un pointeur c'était 4.

Maintenant je recois bien mon SHUTDOWN, et ca me renvoie bien 8 caractères.
Tout fonctionne très bien, merci beaucoup !!

Fred, qui a encore beaucoup à faire pour se dérouiller en C++

**mavina** · 02/10/2009, 09h55

Au fait, comment faire pour que la socket qui écoute écoute toutes les adresses ? actuellement elle refuse toutes les connexions qui ne viennent pas de localhost.

Merci

F.

**nicolas.sitbon** · 02/10/2009, 10h16

Code :

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

sin.sin_addr.s_addr = htonl(INADDR_ANY);

Cordialement.

**mavina** · 02/10/2009, 10h39

Salut,

Merci de ta réponse.

Une derniere chose, mainetnant que je peux recevoir, si j'envoie TEST, et que je compare avec TEST avec strcmp ca me renvoie pas 0.

voici mon nouveau 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
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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
 
#include <cstdlib>
#include <iostream>
#include <winsock2.h>
#pragma comment(lib, "ws2_32.lib")
 
using namespace std;
 
#define buffer_size 1024
 
const char* SHUTDOWN = "SHUTDOWN";
const char* REBOOT = "REBOOT";
const char* LOGOFF = "LOGOFF";
const char* SLEEP = "SLEEP";
 
int main(int argc, char *argv[])
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
 
    SOCKET sock;
    SOCKADDR_IN sin;
    sin.sin_addr.s_addr	= INADDR_ANY;
    sin.sin_family		= AF_INET;
    sin.sin_port		= htons(21);
    sock = socket(AF_INET,SOCK_STREAM,0);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
 
    listen(sock, 0);
    SOCKET val = 0;
 
    int sizeSin = sizeof(sin);
 
    cout << "created socket on port 21" << endl;
    cout << "now listening" << endl;
 
    while(1)
    {
        val = accept(sock, (SOCKADDR *)&sin, &sizeSin);
        if(val != INVALID_SOCKET)
        {
               char* buffer=new char[buffer_size];
               int retour = 0;
               //retour = recv(val,buffer,buffer_size,0 );
               //buffer=new char[buffer_size];
               retour = recv(val,buffer,buffer_size,0 );
               if (retour == SOCKET_ERROR) 
               {
                  cout << "erreur lors de la lecture" << endl << "errno : " << WSAGetLastError() << endl;
                  continue;
               }
               else
               {
                   cout << retour << "caracteres recus" << endl;
                   char* value;
                   strcpy(value,buffer);
                   int i=0;
 
                   cout << "something received" << endl;
 
                   if(strcmp(value,SHUTDOWN)==0)
                   {
                       cout << "SHUTDOWN RECEIVED"<<endl;
                       cout << "SENDING SHUTDOWN"<<endl;
                       system("shutdown -s"); 
                   }
                   else if(strcmp(value,REBOOT)==0)
                   {
                        cout << "REBOOT RECEIVED"<<endl;
                       cout << "REBOOT SHUTDOWN"<<endl;
                       system("shutdown -r"); 
                   }
                   else if(strcmp(value,LOGOFF)==0)
                   {
                       cout << "LOGOFF RECEIVED"<<endl;
                       cout << "LOGOFF SHUTDOWN"<<endl;
                       system("shutdown -l"); 
                   }
                   else if(strcmp(value,SLEEP)==0)
                   {
                       cout << "SLEEP RECEIVED"<<endl;
                       cout << "SLEEP SHUTDOWN"<<endl;
                       system("rundll32.exe powrprof.dll,SetSuspendState"); 
                   }
                   else
                   {
                       cout << "(else) received : " <<value << endl;
                   }
 
               }
 
 
 
        }
    }
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

Et voici le log de mon serveur :

created socket on port 21
now listening
4caracteres recus
something received
(else) received : TESTx☺=
4caracteres recus
something received
(else) received : TESTx☺=
5caracteres recus
something received
(else) received : TESTT☺=
6caracteres recus
something received
(else) received : TESTTT=
7caracteres recus
something received
(else) received : TESTTTT
8caracteres recus
something received
(else) received : TESTTTTT

Pourquoi ces caractères bizarres ?

Merci

F.

**nicolas.sitbon** · 02/10/2009, 10h47

remplace

Code :

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

1
2
char* value;
strcpy(value,buffer);

par

Code :

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

char* value = buffer;

ou bien remplace value par buffer dans tes comparaisons de chaînes.

**ram-0000** · 02/10/2009, 10h50

Parce que tu n'envoie pas le caractère 0 de fin de chaine du côté de ton client (je suppose que tu fais send(socket, string, strlen(string)) ou quelque chose comme cela.

Pour envoyer le 0 de fin de chaine, il faut faire send(socket, string, strlen(string) + 1) ou alors le positionner lors de la reception avec buf_recep[nb_receive] = 0

**mavina** · 02/10/2009, 10h55

Le caractère de fin de chaine c'est 0 ? c'est pas \0 ? Qu'est-ce qu'il se passe si on envoie ton pseudo dans la socket ? Ca marquera juste ram- ?

F.

**Médinoc** · 02/10/2009, 11h15

Cela dépend si l'on suppose comme toi la présence de simple quotes, ou non.
Pour moi et ram-0000, 0 n'est pas '0', et \0 seul ne compile pas. '\0' compile par contre, et est égal à 0.

D'un autre côté, je serais pour ne pas le transmettre, et l'ajouter automatiquement du côté récepteur. Dans ces cas, il peut être intelligent d'envoyer la taille en premier.

**mavina** · 02/10/2009, 12h01

Je vois, merci

Derniere question : comment mettre ce programme en background ?(ce qui est plus pratique pour un daemon) Donc empécher la ligne de commande de s'ouvrir ?

Merci

F.

**nicolas.sitbon** · 02/10/2009, 18h40

Sous Windows, on appelle ça un service (voir MSDN).

**mavina** · 03/10/2009, 05h48

En fait je l'ai mis en service, j'ai crée un service, je l'ai lancé, mais la fenetre revient toujours...

**nicolas.sitbon** · 03/10/2009, 10h05

N'étant pas développeur Windows, je te conseillerais d'ouvrir un nouveau thread dans le forum programmation Windows.
Cordialement.

**mavina** · 04/10/2009, 17h01

Il n'y a donc aucune option de compilation pour ne pas faire apparaitre cette fenetre sous GCC (dev c++)?

Si il n'y en a pas, je posterai sur l'autre forum pour trouver la solution via les services.