Discussion :

[Yann_br]

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
 
// fonction qui créé le tableau 2 D
float** alloc2d(unsigned int nbligne , unsigned int nbcolonne)
{
    float **tab;
    unsigned int i,j;
    if ((nbligne==00) || (nbcolonne==0))
    {
        tab=NULL;
    }
    else
    {
        tab=(float **)malloc(nbligne*sizeof(float *));
        for(i=0; i<nbligne; i++)
        {
            tab[i]=(float *)malloc(nbcolonne*sizeof(float));
            if(tab[i]== NULL)
            {
                //la création du tableau à échoué je dois libérer la mémoire
                for(j=0; j<i; j++)
                {
                    free(tab[j]);
                }
                tab=NULL;
            }
        }
    }
    return tab;
}
 
//programme principal
int main()
{
    int lig,col;
    float ** tableau;
    lig=200000;
    col=200000;
    tableau=alloc2d(lig,col);
    if (tableau==NULL)
    {
        printf("tableau non crée de taille %d %d \n",lig,col);
    }
    return 0;
}

Bonjour,
Comme le tableau est trop grand, j'ai une erreur de segmentation, j'aimerais qu'il m'affiche "tableau non crée de taille ...."

Yann

[Damoun]

Ton alloc est mostrueusement grande :s

Je viens de tester, en mettant 20

Code :

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

1
2
lig=20;
col=20;

Et il n'y a plus de segfault.

[Pouet_forever]

Tu ne testes pas le retour de ton malloc. L'erreur vient peut-être d'ici :

Code :

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

tab=(float **)malloc(nbligne*sizeof(float *));

[Damoun]

j unsigned int 913
i unsigned int 914
tab float ** (float **) 0x0

Il faut verifier le malloc apres la la condition for

Code :

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

1
2
3
4
      for(i=0; i<nbligne; i++)
        {
          if (tab == NULL)
            exit(EXIT_FAILURE);

Pourquoi apres, depassement de la pile d'execution je pense.

[Damoun]

Il faut verifier le malloc apres la la condition for

Code :

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

1
2
3
4
      for(i=0; i<nbligne; i++)
        {
          if (tab == NULL)
            exit(EXIT_FAILURE);

Pourquoi apres, depassement de la pile d'execution je pense.

Ma deuxiemme reponse dois regler sa
Bien sur pour ton cas, tu ne fais pas exit, mais renvois NULL.

Avec cette grosse alloc tu fais une stack overflow.

[Yann_br]

Après correction

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
 
float** alloc2d(unsigned int nbligne , unsigned int nbcolonne)
{
    float **tab;
    unsigned int i,j;
    if ((nbligne==00) || (nbcolonne==0))
    {
        tab=NULL;
    }
    else
    {
        tab=(float **)malloc(nbligne*sizeof(float *));
        for(i=0; i<nbligne; i++)
        {
            if (tab == NULL)
                return NULL;
            tab[i]=(float *)malloc(nbcolonne*sizeof(float));
            if (tab[i]== NULL)
            {
                //la creation du tableau à echoué je dois libérer la mémoire
                for(j=0; j<i; j++)
                {
                    free(tab[j]);
                }
                tab=NULL;
            }
        }
    }
    return tab;
}

Dans mon cas si cela plante pendant la création du tableau, comment être sur que la mémoire utilisé par le tableau est libéré.

[Yann_br]

Ton alloc est mostrueusement grande :s

Je viens de tester, en mettant 20

Code :

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

1
2
lig=20;
col=20;

Et il n'y a plus de segfault.

je sais bien que Alloc2d fonctionne mais je dois traiter tous les type d'appel qui peuvent poser problèmes.
alloc2d(0,10) traité
alloc2d(10,0) traité
alloc2d(100000,100000) segmentation fault
Yann

[Obsidian]

Bonjour,

[code=C]
…
tab=(float **)malloc(nbligne*sizeof(float *));
for(i=0; i<nbligne; i++)
{
tab[i]=(float *)malloc(nbcolonne*sizeof(float));
if(tab[i]== NULL)

Tu testes bien la valeur de retour de tes « sous malloc() » mais pas celle du malloc() principal en début de fonction. Si ton allocation échoue, tu vas provoquer une segfault avant même d'avoir pu remonter jusqu'à main().