Problème de sockets

Version imprimable

Problème de sockets

Bonjour,
Je viens de créer mon premier code utilisant les sockets (presque entièrement pompé sur le tuto de The Walrus) et celui ci ne fonctionne pas :-(
Voici le code client :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <iostream>
#include <winsock2.h>
 
int main()
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKADDR_IN sin;
    char *buffer = new char[255];
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
    sin.sin_family				= AF_INET;
    sin.sin_port				= htons(4148);
    connect(sock, (SOCKADDR *)&sin, sizeof(sin));
    recv(sock, buffer, sizeof(buffer), 0);
    closesocket(sock);
    WSACleanup();
    std::cout << buffer[0] << std::endl;
    system("PAUSE");
}

Et le serveur :

Code:

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
 
#include <winsock2.h>
 
int main()
{
    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(4147);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
    listen(sock, 0);
    for(;;)
    {
        int sinsize = sizeof(csin);
        if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
        {
            send(csock, "Hello world!\r\n", 14, 0);
        }
    }
}

Je lance les 2 programmes sur ma machine (en commençant par le seveur) et la chaine que le client doit afficher est remplacée par d'autres caractères (toujours les mêmes).
Je ne trouve pas mon erreur.
Merci :wink:

12/04/2006, 21h30
Skyounet

Re: Problème de sockets
Citation:
Envoyé par the_penguin

Code:

1 2 std::cout << buffer[0] << std::endl;
C ou C++ à toi de choisir...

TU n'utilises pas les meme ports sur le serveur et le client...
12/04/2006, 21h33
granquet

ce qui serait bien, c'est que ton client se connecte sur le port que le server utilise.
et teste le retour de tes fonctions ... connect etc...
et puis y'as des trucs que je comprend pas ... ca ressemble a du C++.
et pis conserve le type int pour tes sockets, windows en feras pas une maladie, et ce seras plus facilement portable.

edit:: haben grillé ...

et:

Citation:

sock = socket(PF_INET, SOCK_STREAM, 0);
12/04/2006, 23h34
Emmanuel Delahaye

Re: Problème de sockets

Citation:

Envoyé par the_penguin

Voici le code client :

Code:

#include <iostream>

Pas du C. C++, c'est à coté

MODO : A déplacer sur C++. Merci
13/04/2006, 11h17
zul

sizeof(buffer) == sizeof(char*) != 255 ...
Ca devrait résoudre le souci. Cette manie d'utiliser une allocation dynamique pour un buffer de taille fixe ...

Suite à vos conseils, j'ai modifié mon code : J'ai mis le même port pour le client et le serveur et j'ai mis un buffer static pour simplifier.
Voici mon code client :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <stdio.h>
#include <winsock2.h>
 
int main()
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKADDR_IN sin;
    char buffer[255];
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
    sin.sin_family				= AF_INET;
    sin.sin_port				= htons(4147);
    connect(sock, (SOCKADDR *)&sin, sizeof(sin));
    recv(sock, buffer, sizeof(buffer), 0);
    closesocket(sock);
    WSACleanup();
    std::printf("%s", buffer);
    system("PAUSE");
}

Et le serveur :

Code:

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
 
#include <winsock2.h>
 
int main()
{
    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(4147);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
    listen(sock, 0);
    for(;;)
    {
        int sinsize = sizeof(csin);
        if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
        {
            send(csock, "Hello world!\r\n", 14, 0);
        }
    }
}

Mon "Hello World!" s'affiche bien chez le client mais il est suivit de plusieurs caractères indésirables.
De plus, je n'ai pas compris la différence entre AF_INET et PF_INET.

13/04/2006, 19h15
Emmanuel Delahaye
Citation:
Envoyé par the_penguin

Code:

1 2 3 4 sock = socket(AF_INET, SOCK_STREAM, 0); sin.sin_addr.s_addr = inet_addr("127.0.0.1"); sin.sin_family = AF_INET;

De plus, je n'ai pas compris la différence entre AF_INET et PF_INET.
PF_xxx est une constante désignant la Protocol Family. C'est le premier paramètre de socket(). (ici, le type 'Internet', soit Protocole Internet, ou IP)

AF_xxx est une constante désignant la Address Family. C'est la valeur qu'attend le champ sin_family qui concerne l'adressage (ici, de type Internet, c'est à dire IP).
13/04/2006, 19h16
Emmanuel Delahaye
Citation:
Envoyé par the_penguin

Voici mon code client :

Code:

1 2 std::printf("%s", buffer);
Pas du C. le C++, c'est à coté...

Citation:
Envoyé par the_penguin
Voici mon code client :
Code:

1 2 3 4 char buffer[255]; recv(sock, buffer, sizeof(buffer), 0);
Erreur récurrente. Pas de 0 final, pas de place pour le 0 final...
Code:

1 2 3 4 5 6 7 char buffer[255]; int n = recv(sock, buffer, sizeof buffer - 1, 0); if (n >= 0) { char buffer[n] = 0;
Apprenez le C avant de jouer avec les sockets...