Discussion :

[bobgeldof7]

Bonjour,

j ai un petit soucis avec un code c. Le but est en autre d avoir un tableau dynamique liste de type igmp_client et dont le nombre d element est current_host. Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
typedef struct _igmp_client
{
	uchar ip_address[4];
	int interface;
} igmp_client;
 
typedef struct _igmp_entry
{
	uchar groupAddr[4];
	int current_host;
	struct _igmp_client * liste;
 
} igmp_entry;

et dans une fonction, j appelle le code suivant :

Code :

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

1
2
3
4
 
if (igmp_tbl[id].current_host == 0)
           igmp_tbl[id].liste = NULL;
igmp_tbl[id].liste = (igmp_client*) realloc (igmp_tbl[id].liste, (igmp_tbl[id].current_host+1) * sizeof (igmp_client));

et voila l erreur que j obtiens:

Code :

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

1
2
3
 
igmp.c:150: error: incompatible types in assignment
igmp.c:152: error: invalid lvalue in assignment

Si quelqu un pouvait m apporter un petit eclaircisement, ca serait sympa, ca fait tellement longtemps que je n ai pas touche a du C.

Merci bcp.

@++

[Emmanuel Delahaye]

et voila l erreur que j obtiens:

Code :

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

1
2
3
 
igmp.c:150: error: incompatible types in assignment
igmp.c:152: error: invalid lvalue in assignment

Je n'ai pas de problèmes de compilation avec ce code. L'erreur est ailleurs.

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
 
#include <stdio.h>
#include <stdlib.h>
 
typedef unsigned char uchar;
 
typedef struct _igmp_client
{
   uchar ip_address[4];
   int interface;
}
igmp_client;
 
typedef struct _igmp_entry
{
   uchar groupAddr[4];
   int current_host;
   struct _igmp_client *liste;
 
}
igmp_entry;
 
 
int main (void)
{
   igmp_entry igmp_tbl[1];
   int id = 0;
 
   if (igmp_tbl[id].current_host == 0)
      igmp_tbl[id].liste = NULL;
   igmp_tbl[id].liste =
      (igmp_client *) realloc (igmp_tbl[id].liste,
                               (igmp_tbl[id].current_host +
                                1) * sizeof (igmp_client));
 
   return 0;
}

Evidemment, l'usage de realloc() est à revoir, mais c'est un autre problème

http://emmanuel-delahaye.developpez....tes.htm#malloc
http://emmanuel-delahaye.developpez....es.htm#realloc

[Thierry Chappuis]

Salut,

Moi non plus je ne vois aucune erreur dans le code que tu as posté.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
typedef struct _igmp_client
{
	uchar ip_address[4];
	int interface;
} igmp_client;
 
typedef struct _igmp_entry
{
	uchar groupAddr[4];
	int current_host;
	struct _igmp_client * liste;
 
} igmp_entry;

et dans une fonction, j appelle le code suivant :

Code :

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

1
2
3
4
 
if (igmp_tbl[id].current_host == 0)
           igmp_tbl[id].liste = NULL;
igmp_tbl[id].liste = (igmp_client*) realloc (igmp_tbl[id].liste, (igmp_tbl[id].current_host+1) * sizeof (igmp_client)); /* J'utiliserais malloc ici et le cast est déconseillé */

J'ai toutefois une petite remarque sur l'utilsation de realloc. Si igmp_tbl n'est pas NULL mais pointe sur un espace alloué par malloc, et que ton appel à realloc échoue: realloc retournera NULL et l'espace pointé par igmp_tbl ne sera pas libéré (memory leak). Pour une bonne utilisation de malloc, calloc et realloc, voir la doc:
http://man.developpez.com/man3/realloc.3.php
et cet excellent tutoriel:
http://rperrot.developpez.com/articles/c/allocationC/

Thierry

[bobgeldof7]

Evidemment, l'usage de realloc() est à revoir, mais c'est un autre problème

Qu'est ce que tu veux dire ?

[bobgeldof7]

Oups, nos messages se sont croises.

je vais rejeter un coup d oeil a tout ca alors.

Merci de vos reponses.