pb de malloc

Version imprimable

pb de malloc

Bonjour, j'ai un pb de malloc. Voici tout d'abord ce que j'ai fait :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
typedef struct t_mat
{
    int dim;
    double * data;
} Tableau;
 
 
void alloc_Tableau(Tableau *t,unsigned int dim_)
{
    assert(dim_>0);
    t->dim=dim_;
    t->data=malloc(dim_*sizeof(double));
    assert(t->data!=NULL);
    int i;
    for(i=0;i<t->dim;i++)
	t->data[i]=0.;
    return ;
}
 
void free_Tableau(Tableau * t)
{
    free(t->data);
    t->data=NULL;
    return ;
}
 
void affiche_Tableau(Tableau t)
{
    if(t.data==NULL)
	printf("\nTableau inexistant\n");
    else
    {
	int i;
	for(i=0;i<t.dim;i++)
	    printf("%3.5g\t",t.data[i]);
	printf("\n");
    }
 
    return;
}

Dans le main je fais la chose suivante :

Code:

1
2
3
4
5
 
Tableau t1;
alloc_Tableau(&t1,3);
affiche_Tableau(t1);
free_Tableau(&t1);

et là j'obtiens bien la sortie

Code:

Maintenant si je fais

Code:

1
2
3
 
Tableau * t2
alloc_Tableau(t2,3);

et bien j'ai une erreur de segmentation. Savez-vous pourquoi ?

Merci beaucoup.

04/11/2005, 12h22
khayyam90

Re: pb de malloc
bien le bonjour,
Citation:
Envoyé par salseropom

[...]
Maintenant si je fais

Code:

1 2 3 Tableau * t2 alloc_Tableau(t2,3);

et bien j'ai une erreur de segmentation. Savez-vous pourquoi ?
tu dois allouer de la memoire pour y stocker ton 'Tableau'.
il te manque un t2 = malloc ...
sinon, tu ecris dans un emplacement non alloue.
04/11/2005, 12h25
salseropom

Salut khayyam90,

Citation:

il te manque un t2 = malloc ...

mais c'est ce que je fais dans ma fonction alloc_Tableau (enfin c'est ce que je voudrais faire). Où est donc l'erreur dans ma fonction car visiblement elle marche pour un Tableau t1 mais pas pour un Tableau * t2

Merci
04/11/2005, 12h36
Emmanuel Delahaye
Citation:

Envoyé par salseropom

Citation:

il te manque un t2 = malloc ...

mais c'est ce que je fais dans ma fonction alloc_Tableau (enfin c'est ce que je voudrais faire). Où est donc l'erreur dans ma fonction car visiblement elle marche pour un Tableau t1 mais pas pour un Tableau * t2
- Passer une valeur indéfinie à une fonction invoque un comportement indéfini. C'est un bug.
- Modifier un paramètre dans une fonction est souvent un erreur de conception. La valeur initiale (de l'appelant) restera inchangée.
Autrement dit, ta démarche est erronée.

Si tu veux qu'une fonction crée une variable, le plus simple est de faire comme malloc() : retourner l'adresse de la variable créée.
04/11/2005, 12h37
khayyam90

Citation:

Envoyé par salseropom

mais c'est ce que je fais dans ma fonction alloc_Tableau (enfin c'est ce que je voudrais faire)

non, dans ta fonction alloc_Tableau tu alloues de la memoire pour t->data, encore faudrait-il que t->data soit defini, ce qui n'est pas le cas si tu prends un Tableau * t2 .
04/11/2005, 12h40
jack69

Citation:

mais c'est ce que je fais dans ma fonction alloc_Tableau (enfin c'est ce que je voudrais faire). Où est donc l'erreur dans ma fonction car visiblement elle marche pour un Tableau t1 mais pas pour un Tableau * t2

Non ta fonction alloue l'espace du "tableau" de double défini dans ta structure.
La tu crées un pointeur vers ta structure. Il faut donc l'allouer pour ensuite appeler ta fonction qui feras l'allocation du tableau.

C'est pas très clair hein ?

Re-bonjour, merci à vous, j'ai compris !

Voici donc ce que j'ai fait :
Code:

1 2 3 4 5 6 7 8 9 10 11 Tableau t1; alloc_Tableau(&t1,3); affiche_Tableau(t1); free_Tableau(&t1); Tableau * t2; t2=malloc(sizeof(Tableau)); alloc_Tableau(t5,4); affiche_Tableau(*t5); free_Tableau(t5);
Une dernière question : est-ce que mon t5 est bien effacé de la mémoire ou est-ce seulement le t5->data qui est effacé ?
Ai-je une fuite de mémoire à la fin de mon prgm (avec le free_Tableau(t5)) ?

Merci encore.

04/11/2005, 13h35
salseropom

excusez -moi, lisez t2 au lieu de t5
04/11/2005, 13h43
khayyam90

Citation:

est-ce que mon t5 est bien effacé de la mémoire ou est-ce seulement le t5->data qui est effacé ?

non, t2 n'est pas libéré. tu as bien libéré t2->data via free_Tableau, mais pas t2
04/11/2005, 14h02
salseropom

aïe, je m'en doutais car j'avais deux malloc et un seul free. Et comment dois-je libérer t2 ?
que dois-je taper free(??) ?

Merci beaucoup de votre patience.
04/11/2005, 14h10
DaZumba

Ben puisque tu fais t2 = malloc(), il faut faire le free(t2) qui va avec.
C'est vraiment simple: a un malloc() doit correspondre un free().
04/11/2005, 14h16
salseropom

oups, effectivement, c'était un peu stupide comme question.

Merci de votre patience et de vos explications en tout cas.