return et exit, comment les utiliser proprement ? + parsage d'adresse ethernet

Version imprimable

Voir 40 message(s) de cette discussion en une page

Voilà le résultat :

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
int
send_packet (int sockfd, const void *buff, size_t const bufflen, const struct sockaddr *to, socklen_t const tolen)
{
    size_t   sent = 0;
    ssize_t  n = 0;
    int      err = 0;
 
    while (sent != bufflen)
    {
        /* n = sendto (sockfd, buff, bufflen, 0, to, tolen); */
        n = sendto (sockfd, buff + sent, bufflen - sent, 0, to, tolen);
 
        if (n >= 0)
            sent += n;
        else
        {
            /* On force la sortie */
            sent = bufflen;
            err = 1;
        }
    }
 
    return err;
}

Citation:

Envoyé par Freed0

Voilà le résultat :

J'ai bien peur que tu envoies toujours la même chose... Il faut progresser, donc envoyer 'buff + sent' (et corriger ce qu'il reste à envoyer)... non ?

C'est pas ce que j'avais mis dans mon algo ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 /* non teste */ size_t const len = strlen (text); size_t sent = 0; while (sent != len) { int n = send (socket, text + sent, len - sent, 0); if (n >=0) { sent += n; } else { /* erreur de transmission */ } }
A quoi ca sert que je donne des réponses si on en tient pas compte ? Il y a des choses que je ne comprendrais jamais...

20/05/2007, 14h55
Freed0

Roh comment j'ai pu oublier ça :aie:
20/05/2007, 15h13
Freed0
Une idée pour éviter ceci ?
Code:

1 2 src/main.c: In function 'send_packet': src/main.c:193: warning: pointer of type 'void *' used in arithmetic
Code:

93: n = sendto (sockfd, buff + sent, bufflen - sent, 0, to, tolen);

On peut pas recaster ça :? Donc je vois pas trop ce qui nous permettrait de modifier l'adresse de départ lors de l'envoi...
20/05/2007, 15h49
Emmanuel Delahaye
Citation:
Envoyé par Freed0

Une idée pour éviter ceci ?

Code:

1 2 src/main.c: In function 'send_packet': src/main.c:193: warning: pointer of type 'void *' used in arithmetic

Code:

93: n = sendto (sockfd, buff + sent, bufflen - sent, 0, to, tolen);

On peut pas recaster ça :? Donc je vois pas trop ce qui nous permettrait de modifier l'adresse de départ lors de l'envoi...
buff devrait être de type unsigned char *, sinon utiliser un pointeur local de ce type...
Code:

1 2 3 4 5 6 ... f (..., void *buff, ...) { unsigned char *o_buff = buff; n = sendto (sockfd, o_buff + sent, bufflen - sent, 0, to, tolen);
20/05/2007, 15h52
Freed0

J'y ai pensé, mais si on veut envoyer autre chose que des octets non-signés ?
20/05/2007, 16h35
Emmanuel Delahaye

Citation:

Envoyé par Freed0

J'y ai pensé, mais si on veut envoyer autre chose que des octets non-signés ?

Je ne pense pas qu'un socket sache envoyer autre chose...

Bon, pour ne pas créer de nouveau sujet à chaque fois, j'ai une petite question sur le cast :

J'ai ceci :

Code:

1
2
3
4
5
6
7
8
9
struct icmp      icmp = {0};
pid_t            pid = getpid();
unsigned short   sent = 0;
 
icmp.icmp_type  = ICMP_ECHO;
icmp.icmp_code  = 0;
icmp.icmp_id    = pid;
icmp.icmp_seq   = sent++;
icmp.icmp_cksum = 0;

Sachant que la structure icmp est composée comme ceci :

Code:

1
2
3
4
5
6
7
8
9
    0                   1                   2                   3
    0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
   |     Type      |     Code      |          Checksum             |
   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
   |           Identifier          |        Sequence Number        |
   +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
   |     Data ...
   +-+-+-+-+-

Faut-il caster le Process ID pour le placer dans le membre icmp_id ?

21/05/2007, 12h12
Skyounet

Citation:

Envoyé par Freed0

Faut-il caster le Process ID pour le placer dans le membre icmp_id ?

Pourquoi voudrais-tu le caster ?
21/05/2007, 12h22
Freed0

Bah pid_t occupe 4 bytes et icmp_id en occupe 2...
21/05/2007, 12h37
Skyounet

Citation:

Envoyé par Freed0

Bah pid_t occupe 4 bytes et icmp_id en occupe 2...

Ben moi je dis que tu n'as pas besoin de caster.
21/05/2007, 12h38
Freed0

Citation:

Envoyé par Skyrunner

Ben moi je dis que tu n'as pas besoin de caster.

C'est ça que je voulais savoir. Merci ;)
21/05/2007, 13h01
Médinoc

N'est-on pourtant pas supposé caster quand on copie d'un type entier vers un type entier plus petit ? 8O
21/05/2007, 13h12
Freed0

Je suis tombé sur un tableau assez interessant :

http://docs.hp.com/en/B3901-90003/ch...sting-integers

Apparament une conversion de type int vers short serait dans la catégorie C :

Citation:

CASE C: Casting Integers to a Smaller Type

When an int value is cast to a narrower type (short or char), the excess bits on the left are discarded. The same is true when a short is cast to a char, or when a long in 64-bit mode is cast to an int. For instance, if an int is cast to a short, the 16 leftmost bits are truncated. The following table of values illustrates these conversions.

...

If, after casting to a signed type, the leftmost bit is 1, then the number is negative. However, if you cast to an unsigned type and after the shortening the leftmost bit is 1, then that 1 is part of the value (it is not the sign bit).

Etant donné que dans mon cas, je cast un int en unsigned short, il n'y a rien à faire. Mais si cela aurait été un signed short, il aurait fallu le caster en unsigned short... Enfin c'est ce que j'ai compris du texte :)
21/05/2007, 13h18
Médinoc

Hélas, ça ne dit pas si la conversion est implicite ou doit être explicite... :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <stdio.h>
 
int
main (void)
{
	int		pid = 80000;
	unsigned short	s_pid = 0;
 
	printf("pid (int) : %d (0x%08x)\n", pid, pid);
 
	s_pid = pid;
	printf("pid (no cast) : %hd (0x%x)\n", s_pid, s_pid);
 
	s_pid = (unsigned short) pid;
	printf("pid (cast) : %hd (0x%x)\n", s_pid, s_pid);
 
	s_pid = pid & 0xffff;
	printf("pid (mask) : %hd (0x%x)\n", s_pid, s_pid);
 
	return 0;
}

Résultat :

Code:

1
2
3
4
pid (int) : 80000 (0x00013880)
pid (no cast) : 14464 (0x3880)
pid (cast) : 14464 (0x3880)
pid (mask) : 14464 (0x3880)

De toute façon, je devrai quand même caster le Process ID quand je récupèrerai la réponse ICMP, pour pouvoir le comparer avec la valeur reçue.

21/05/2007, 13h38
Médinoc

Et pas un warning sans le cast ?
OK, la norme de doit sans-doute pas l'exiger, les warnings doivent donc être fournis pas le compilo en tant qu'extensions...
21/05/2007, 14h10
Emmanuel Delahaye

Citation:

Envoyé par Médinoc

N'est-on pourtant pas supposé caster quand on copie d'un type entier vers un type entier plus petit ? 8O

C'est pas obligatoire. Le cast implicite suffit, mais ça peut faire raler le compilateur ("Eh, le programmeur, es tu bien sûr de ce que tu fais ?").

Par contre, oui, il y a risque de perte de données si le short est plus petit que l'int (ce qui n'a rien d'une obligation).
21/05/2007, 20h22
Freed0

Tiens, pour éxécuter du code tous les x temps, y'a quoi de bien ?
En décortiquant la source d'un soft, j'ai vu qu'on pouvait utiliser SIGALRM. Y'a quoi d'autres ? C'est pour envoyer une requête ICMP toutes les secondes en gros.
Pour la réception et les time out, je pense que select() sera le plus adapté. D'autres idées ?

Merci
21/05/2007, 20h31
Skyounet

Citation:

Envoyé par Freed0

Tiens, pour éxécuter du code tous les x temps, y'a quoi de bien ?
En décortiquant la source d'un soft, j'ai vu qu'on pouvait utiliser SIGALRM. Y'a quoi d'autres ? C'est pour envoyer une requête ICMP toutes les secondes en gros.
Pour la réception et les time out, je pense que select() sera le plus adapté. D'autres idées ?

Merci

Tu veux que ça coupe ce que tu es en train de faire pour faire le ping ? Une sorte d'interruption ??

Voir 40 message(s) de cette discussion en une page