Discussion :

[humitake]

Bonjour,

Je souhaiterais pouvoir ajouter un timout sur le connect de ma socket. J'ai déjà un timout effectif pour l'envoie mais malgré plusieurs essaie et des recherches sur internet impossible de trouver une solution.

Voici le bout de code de la fonction que j'utilise :

Code c++ :

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
SOCKET sock; //Cr�ation de la socket
SOCKADDR_IN sin;
int size = 10;
char *buffer;
buffer = (char*) malloc(sizeof(char*) * size);
memset(buffer, 0, size);
fd_set readfs;
struct timeval tv;
 
sin.sin_addr.s_addr = inet_addr(ip);
sin.sin_family      = AF_INET;
sin.sin_port        = htons(502);
sock = socket(AF_INET,SOCK_STREAM,0);
bind(sock, (SOCKADDR *)&sin, sizeof(sin));
 
 
if(connect(sock, (SOCKADDR *)&sin, sizeof(sin)) == SOCKET_ERROR)
    return "-1";
 
int ret = 0;
FD_ZERO(&readfs);
FD_SET(sock, &readfs);
tv.tv_sec = 5;
tv.tv_usec = 0;
 
buffer[0] = 0x00; //Transaction ID
buffer[1] = 0x01; //Transactio6n ID
buffer[2] = 0x00; //Protocole ID
buffer[3] = 0x00; //Protocole ID
buffer[4] = 0x00; //Length
buffer[5] = 0x04; //Length
buffer[6] = numEsclave; //Num�ro d'esclave
buffer[7] = 0x11; //Num�ro de la fonction
//Envoie de la socket
int a = send(sock, //La socket � envoyer
                    buffer, //La requete � envoyer
                    size, //La taille de la requete
                    0);
if(a < 0)
    return "-1";

Le but serais donc de gérer un timeout d'environ 5 secondes sur la fonction connect se trouvant ligne 17 et qu'en cas de timeout cela retourne -1.
Est-ce que quelqu'un pourrait me donner une piste pour gérer un timeout sur une fonction connect ?
Je vous remercie d'avance

[phi1981]

t'as essayé setsockopt() ??? tu as SND_TIMEO et RCV_TIMEO qui contrôlent le time out sur un send ou un recv, c'est pas impossible que ça gère aussi celui du connect. à essayer.

sinon rien à voir : fais gaffe à l'allocation de ton buffer : (char*) malloc(sizeof(char*) * size);
là tu alloues 4 fois la taille "size", je ne sais pas si ce que tu voulais vraiment faire.

mmm ... TransactionID, numEsclave, .... serait-ce du modbus ????

[humitake]

Super avec setsockopt() ça marche nikel. J'ai juste un peu modifier mon code pour pouvoir réutiliser le même timer que pour mon select : (le passage intéressant est ligne 23 )

Code c++ :

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
SOCKET sock; //Création de la socket
SOCKADDR_IN sin;
int size = 10;
char buffer[size];
memset(buffer, 0, size);
fd_set readfs;
struct timeval tv;
 
sin.sin_addr.s_addr = inet_addr(ip);
sin.sin_family      = AF_INET;
sin.sin_port        = htons(502);
sock = socket(AF_INET,SOCK_STREAM,0);
bind(sock, (SOCKADDR *)&sin, sizeof(sin));
 
 
int ret = 0;
FD_ZERO(&readfs);
FD_SET(sock, &readfs);
tv.tv_sec = 5;
tv.tv_usec = 0;
 
setsockopt(sock, SOL_SOCKET, SO_SNDTIMEO, (void *)&tv, sizeof(tv));
if(connect(sock, (SOCKADDR *)&sin, sizeof(sin)) == SOCKET_ERROR)
    return "-1";
 
buffer[0] = 0x00; //Transaction ID
buffer[1] = 0x01; //Transactio6n ID
buffer[2] = 0x00; //Protocole ID
buffer[3] = 0x00; //Protocole ID
buffer[4] = 0x00; //Length
buffer[5] = 0x04; //Length
buffer[6] = numEsclave; //Numéro d'esclave
buffer[7] = 0x11; //Numéro de la fonction
// Pas de CRC 16 dans ce mode de contact !
//Envoie de la socket
int a = send(sock, //La socket à envoyer
         buffer, //La requete à envoyer
         size, //La taille de la requete
         0);
if(a < 0)
    return "-1";

sinon rien à voir : fais gaffe à l'allocation de ton buffer : (char*) malloc(sizeof(char*) * size);
là tu alloues 4 fois la taille "size", je ne sais pas si ce que tu voulais vraiment faire.

Effectivement merci je devais être très fatigué ce jour la, en plus il existe plus simple char buffer[size];

mmm ... TransactionID, numEsclave, .... serait-ce du modbus ????

Quel oeil de lynx !
C'est effectivement du modbus, j'utilise dans le reste du programme la bibliothèque libmodbus mais c'est ici une fonction spécifique au constructeur que j'utilise et elle n'est donc pas implémenté dans la biblio.

Merci beaucoup pour le coup de main

[NazOok]

Bonjour

Désolé pour la question que je vais poser mais je ne suis vraiment pas alaise avec le C/C++..

C'est effectivement du modbus, j'utilise dans le reste du programme la bibliothèque libmodbus mais c'est ici une fonction spécifique au constructeur que j'utilise et elle n'est donc pas implémenté dans la biblio.

Pour la blibliothèque Libmodbus, c'est une bibliothèque en C principalement porté sur linux.. Je vois que ton code est du C++ et je me demande comment tu as utilisé cette librairie ? On peut directement utiliser une librairie en C dans du code C++ ? ou bien il faut développer un wrapper ?

Cordialement,

NazOok.

[dragonjoker59]

Le C++ est fait de manière à être compatible avec le C : tu peux compiler du code C avec un compilateur C++ (la réciproque n'est pas vraie).
Tu peux donc utiliser sans soucis une bibliothèque écrite en C dans du code C++, c'est même très courant en fait. (il suffit de considérer qu'OpenGL est une lib C)

[NazOok]

Le C++ est fait de manière à être compatible avec le C : tu peux compiler du code C avec un compilateur C++ (la réciproque n'est pas vraie).
Tu peux donc utiliser sans soucis une bibliothèque écrite en C dans du code C++, c'est même très courant en fait. (il suffit de considérer qu'OpenGL est une lib C)

Oui en effet c'est ce que j'ai appris dans mes dernières recherches tout à l'heure
Maintenant reste plus qu'à réussir à compiler la bibliothèque de libmodbus, je galèreeeeeeeeee ^_^