faute de segmentation en cours d'allocation dynamique de la mémoire avec malloc

Cela peut être très idiot de se poser la question.

Je suis à l'aide de malloc pour l'allocation de mémoire.

Programme compile bien, mais faute de segmentation se produit lors du lancement.

Voici min code: 

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main()
{
    int row,col,i;
    int **mat;
    row = 3; //made row as static for testing
    col = 4; //made col as static for testing
    *mat = (int *)malloc(sizeof(int)*row);
    for (i=0;i<row;i++)
       mat[i] = (int *) malloc (sizeof(int)*col);
}

j'ai compilé avec: gcc-ggdb test.c

sur gdb sa présentation: 

(gdb) run
Starting program: /slowfs/swe91/abjoshi/clients/fw_cl_for_seg_fault/a.out

Program received signal SIGSEGV, Segmentation fault.
0x00000000004004cc in main () at 1test.c:10
10          *mat = (int *)malloc(sizeof(int)*row);

Note: gcc version 4.5.2

  • *mat = (int *)malloc(sizeof(int)*row); tapis n'est pas pour tout de suite.
  • c'est ce que je suis affectation de la mémoire de pointeur, alors je vais stocker des valeurs dans cet emplacement de la mémoire. S'il vous plaît laissez-moi savoir si je suis absent quelque chose.
  • Voir ma réponse ci-dessous.
InformationsquelleAutor Abhi | 2013-09-18
  1. 7

    La commune malloc idiome est le suivant: sortie d'allouer un tableau de n éléments pointée par le pointeur p utilisation suivantes universelle syntaxe

    p = malloc(n * sizeof *p);

    Qui vous aidera à éviter les erreurs, comme celle de votre code. Notez les points clés ici: ne lancez pas le résultat de malloc et n'utilisez pas les types de sous sizeof. Et en général: autant que possible, éviter de mentionner les types de n'importe où en dehors des déclarations.

    Dans votre cas, la première allocation devrait ressembler

    mat = malloc(row * sizeof *mat);

    Les affectations à l'intérieur du cycle devrait ressembler

    for (i = 0; i < row; ++i)
   mat[i] = malloc(col * sizeof *mat[i]);

    L'avantage important de cette approche est que le code est de type indépendant: à tout moment, vous pouvez modifier la déclaration de mat de int **mat à, disons, double **mat, mais de la répartition de code continuera d'être valable - vous n'aurez pas à effectuer des changements.

    • merci beaucoup c'est vraiment une grande explication.
    InformationsquelleAutor AnT
  2. 3

    Au lieu de 

    *mat = (int *)malloc(sizeof(int)*row);

    Je pense que vous voulez 

    mat = (int **)malloc(sizeof(int *) * row);

    Comme il est, vous êtes un déréférencement de tapis qui ne sont pas initialisés. Vous avez besoin de malloc() mat avant de déférence il.

    Vous pouvez laisser la fonte de la malloc ainsi:

    mat = malloc(sizeof(int *) * row);

    Ainsi que la sizeof(int *) référence:

    mat = malloc( row * sizeof(*mat));
    InformationsquelleAutor Charlie Burns
  3. 2

    Compilateur aurait dit il avec -Wall

    $ gcc -Wall y.c
y.c: In function ‘main’:
y.c:14:1: warning: control reaches end of non-void function [-Wreturn-type]
y.c:11:8: warning: ‘mat’ is used uninitialized in this function [-Wuninitialized]
    InformationsquelleAutor Victor von Cacahuete
  4. 1

    Dans la ligne *mat = (int *) malloc(sizeof(int)*row); vous essayez de déréférencement de tapis. mat est actuellement non initialisée.

    InformationsquelleAutor Andy
  5. 0

    Voici ce qu':

    mat est un double pointeur et il est non initialisée (ou NULL).
    Lorsque vous déréférencement d'elle:

    *mat = (int *)malloc(sizeof(int)*row);

    vous êtes à la référence à une valeur NULL et en essayant de les attribuer à elle, ce qui provoque l'erreur de segmentation.

    D'abord l'initialiser tapis avec un pointeur de liste (int*) les pointeurs.
    Ensuite, individuellement, chaque pointeur dans la liste et attribuer (int) de l'espace pour chacun.

    mat = (int *)malloc(cols*sizeof(int *));
for (i=0;i<cols;i++)
    mat[i] = malloc(rows*sizeof(int))
    InformationsquelleAutor AbhinavChoudhury