Discussion :

[The Free Man]

Bonjour tout le monde!
Voila dans mon programme j'ai un sendto qui envoi une requete à un serveur.
Jusque la mon sendto fonctionnait bien mais quand j'ai paramétré le port udp, sendto() me renvoi cette erreur : sendto() : invalid argument.

Quand je rentre le port en dur comme ici : si_other.sin_port = htons(1903); tout fonctionne correctement. En vérité, mon port est contenu dans un fichier texte. Donc je vais le chercher, je le retransforme en int grâce à la fonction atoi(), je l'affiche et j'affiche exactement ce qu'il me faut. Par contre quand je le passe dans le htons, quand je compile le sendto n'est pas content.

Quelqu'un aurait-il une idée ?
Merci

[nicolas.sitbon]

On peut voir le code?

[The Free Man]

Ui le voila :p

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
 
#include ....
 
char texteudpport[]="Numeros du port UDP :";
char *strportudp=NULL;
 
int main(void)
{
        int udpport;
	struct sockaddr_in si_other;
	int s, i, slen=sizeof(si_other);
 
strportudp = RecupChemin(texteudpport);
udpport=atoi(strportudp);
 
if ((s=socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP))==-1)
          diep("socket");
 
        memset((char *) &si_other, 0, sizeof(si_other));
        si_other.sin_family = AF_INET;
        si_other.sin_port = htons(udpport);
        if (inet_aton(ipcentrale, &si_other.sin_addr)==0) {
          fprintf(stderr, "inet_aton() failed\n");
          exit(1);
        }
sendto(s, trame, 29, 0, (struct sockaddr *)&si_other, slen)==-1
..
...
}

La fonction RecupChemin retourne un char*: elle va chercher dans le fichier le numero du port udp

[ram-0000]

J'ai l'impression que tu utilises mal inet_aton() et donc que la structure si_other est mal remplie

[The Free Man]

Ah bon ?
Alors c'est surement pour ça que ça foire :/
Mais le truk c'est que sa marche si je rentre le numero en dur et ça marche aussi si je fais ça :

Code :

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

1
2
3
4
5
6
7
 
int numport;
 
numport = 1903;
 
...
si_other.sin_port = htons(numport);

Que je fasse ça ou numport = atoi(char*); c'est pareil (nan?) car de toutes façons c'est stocké dans un int et quand je l'affiche j'ai le bon numero

[ram-0000]

Je pense que cela marche un peu par hazard.

Si on regarde l'aide de inet_aton() qui au passage est une fonction qu'il ne faudrait plus utiliser, le 2eme paramètre doit être l'adresse d'une structure struct in_addr.

Dans ton code, tu lui passes l'adresse du membre sin_addr de cette structure struct in_addr. Il y a donc de fortes chances que cela ne soit pas bon.