Discussion :

[Marnage]

Bonjour,

Voila je fais un petit code sur les raw sockets et voila , a la compilation , tous va bien seulement sa ne plait pas a valgrind qui me detecte quelque chose que je ne comprend pas !

==8180== Syscall param socketcall.sendto(msg) points to uninitialised byte(s)
==8180== at 0x4000792: (within /lib/ld-2.3.6.so)
==8180== by 0x8049133: fn_thread_scan (in /home/marnage/s)
==8180== by 0x402E23F: start_thread (in /lib/tls/i686/cmov/libpthread-2.3.6.so)
==8180== by 0x410749D: clone (in /lib/tls/i686/cmov/libc-2.3.6.so)
==8180== Address 0x416E958 is 32 bytes inside a block of size 40 alloc'd
==8180== at 0x401C38B: malloc (vg_replace_malloc.c:149)
==8180== by 0x8049A04: init_struct (in /home/marnage/s)
==8180== by 0x804918A: scan_port (in /home/marnage/s)
==8180== by 0x8049133: fn_thread_scan (in /home/marnage/s)
==8180== by 0x402E23F: start_thread (in /lib/tls/i686/cmov/libpthread-2.3.6.so)
==8180== by 0x410749D: clone (in /lib/tls/i686/cmov/libc-2.3.6.so)

Donc ont peut voir
==8180== at 0x401C38B: malloc (vg_replace_malloc.c:149)
==8180== by 0x8049A04: init_struct (in /home/marnage/s)

Je dois comprendre qu'un malloc() dans cette fonction n'et pas bon?
Voila la fonction :

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
int
init_struct(packet_t **packet)
{
	*packet = NULL;
 
	*packet = (packet_t *) malloc(sizeof(packet_t));
	if ( *packet == NULL ) {
		fprintf(stderr, "malloc() on packet failed\n");
		return -1;
	}
 
	(*packet)->tcp = NULL;
	(*packet)->ip = NULL;
	(*packet)->fakehdr = NULL;
	(*packet)->pkt_send = NULL;
	(*packet)->pkt_recv = NULL;
 
	(*packet)->pkt_send = (char *) malloc(LEN_PACKET);
	if ( (*packet)->pkt_send == NULL ) {
		fprintf(stderr, "malloc() on pkt_send failed\n");
		return -1;
	}
 
	(*packet)->ip = (iphdr_t *) (*packet)->pkt_send;
	if ( (*packet)->ip == NULL ) {
		fprintf(stderr, "ip value error\n");
		return -1;
	}
 
	(*packet)->tcp = (tcphdr_t *) ((*packet)->pkt_send + sizeof(iphdr_t));
	if ( (*packet)->tcp == NULL ) {
		fprintf(stderr, "ip value error\n");
		return -1;
	}
 
	(*packet)->fakehdr = (fakehdr_t *) malloc(sizeof(fakehdr_t));
	if ( (*packet)->fakehdr == NULL ) {
		fprintf(stderr, "malloc() on fakehdr failed\n");
		return -1;
	}
 
	return 0;
}

Personellement je ne vois pas se que valgrind lui trouve ... si quelqu'un aurait une piste ...

Voici la fonction sendto() a tout hasard :

Code :

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

1
2
 
	if ( sendto(sock->s, packet->pkt_send, LEN_PACKET, 0, (struct sockaddr *)&sock->to, sizeof(sock->to)) == -1 )

Et le paquet une fois envoyer est correcte ... .
Merci.
++

[Obsidian]

Personellement je ne vois pas se que valgrind lui trouve ... si quelqu'un aurait une piste ...
Merci.
++

Il y a deux messages différents dans ton log de Valgrind, mais avant toute chose, laquelle des lignes que tu nous a montrées est la numéro 149 ?

[Marnage]

La ligne 149 de mon code :

Code :

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

1
2
 
void*		fn_thread_func(void *argument);

[Marnage]

Ok ba je marque résolu j'ai trouver le problème .
Merci encore et dsl pour le flood

[Obsidian]

Ok ba je marque résolu j'ai trouver le problème .
Merci encore et dsl pour le flood

Ok, fais-en profiter les autres, maintenant :-)

[Emmanuel Delahaye]

Personellement je ne vois pas se que valgrind lui trouve ... si quelqu'un aurait une piste ...

Je ne vois rien d'horrible (à part des cast douteux et non portables), mais passer l'adresse d'un pointeur rend le code inutilement complexe.

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
 
#include <stdio.h>
#include <stdlib.h>
 
#define LEN_PACKET 1024
 
typedef struct
{
   int dummy;
}
iphdr_t;
 
typedef struct
{
   int dummy;
}
tcphdr_t;
 
typedef struct
{
   int dummy;
}
fakehdr_t;
 
typedef struct
{
   tcphdr_t *tcp;
   iphdr_t *ip;
   fakehdr_t *fakehdr;
   unsigned char *pkt_send;
   unsigned char *pkt_recv;
}
packet_t;
 
packet_t *init_struct (void)
{
   packet_t *packet = malloc (sizeof (packet_t));
 
   if (packet != NULL)
   {
      packet->tcp = NULL;
      packet->ip = NULL;
      packet->fakehdr = NULL;
      packet->pkt_recv = NULL;
 
      packet->pkt_send = malloc (LEN_PACKET);
 
      if (packet->pkt_send != NULL)
      {
         packet->ip = (iphdr_t *) packet->pkt_send;
         packet->tcp = (tcphdr_t *) (packet->pkt_send + sizeof (iphdr_t));
 
         packet->fakehdr = malloc (sizeof (fakehdr_t));
         if (packet->fakehdr == NULL)
         {
            fprintf (stderr, "malloc() on fakehdr failed\n");
         }
      }
      else
      {
         fprintf (stderr, "malloc() on pkt_send failed\n");
      }
   }
   else
   {
      fprintf (stderr, "malloc() on packet failed\n");
   }
   return packet;
}
 
int main (void)
{
   packet_t *packet = init_struct ();
 
   return 0;
}

[Marnage]

Ok, fais-en profiter les autres, maintenant :-)

Donc en faite le problème a était regler avec un

Code :

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

memset((*packet)->pkt_send, 0, LEN_PACKET);

Après avoir allouer l espace mémoire de packet->pkt_send

@ Emmanuel Delahaye :
Ton code je le comprend pas vraiment , tu ne donne pas d argument a ta fonction ,

Code :

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

packet_t *init_struct (void)

Hum je vois un void moi , si tu aurais un tuto qui parle de sa car la je suis largué et utiliser un truc que je comprend pas bof .
En tous cas merci .

[diogene]

void (mais pas void*) dans la liste des paramètres d'une fonction signifie que la fonction ne prend pas d'argument.

Ton code passe l'adresse d'un pointeur dans lequel tu places l'adresse de la mémoire allouée et tu utilises la valeur de retour comme code d'erreur.

Le code d'Emmanuel renvoie directement l'adresse de la mémoire allouée.
La gestion d'erreur d'allocation de packet se fait sur cette valeur : erreur si NULL, pas d'erreur sinon.
La gestion d'erreur (si packet est alloué) sur packet->pkt_send ou sur packet->fakehdr se fait sur la valeur NULL ou non stockée dans ces pointeurs