Segfault lors de la lecture des données du fichier pour réallouer le tableau struct struct

Question

J'essaie de lire les données d'un fichier dans un tableau struct struct, la taille du tableau étant étendue via realloc pour prendre en compte des données supplémentaires.

Il semble fonctionner pour une seule ligne de données (taille de tableau 1), mais rien de plus gros et je reçois un segfault. Je suis assez inexpérimenté au codage, donc je suis complètement perdu sur ce qui le cause.

Voici le code correspondant:

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

typedef struct 
{ 
    int a; 
    int b; 
} Structure; 

void Tester(Structure **Data) 
{ 
    Structure Input; size_t Number = 0; 

    *Data = NULL; 

    FILE *File = fopen("file.txt", "r"); 

    while(2 == fscanf(File, "%d %d", &Input.a, &Input.b)) 
    { 
     printf("Before: %d\n",Number); 

     Number++; 

     Structure *NewInput = realloc(*Data, Number * sizeof(Structure)); 

     *Data = NewInput; 

     printf("After: %d\n",Number); 

     *Data[Number - 1] = Input; 
    } 

    fclose(File); 
} 

int main() 
{ 
    Structure *TestData; 

    Tester(&TestData); 

    printf("%d %d", TestData[0].a, TestData[0].b); 

    return 0; 
} 

Le fichier texte que j'ai utilisé ne contenait que « 1 2 » lorsque le programme a fonctionné, et « 1 2 3 4 » quand il ne l'a pas. Les occours d'erreur à la ligne:

 *Data[Number - 1] = Input; 

Je le sentiment que le realloc ne fonctionne pas correctement après la première boucle. Toute aide ou conseil sera grandement apprécié!

Answer 1

En raison de la priorité des opérateurs

*Data[Number - 1] = Input; 

est équivalent à

*(Data[Number - 1]) = Input; 

Cette traite Data comme un tableau de pointeurs, quand il est en fait un seul pointeur vers un autre pointeur qui pointe vers un tableau de Structure. Il utilise ensuite le Nième élément de ce tableau (qui n'existe pas réellement) comme emplacement pour copier Input vers. Il échoue quand Number > 0 car aucun de ces autres pointeurs n'a été initialisé (puisqu'il n'y a aucun tableau réel là).

La syntaxe correcte est:

(*Data)[Number - 1] = Input; 

Ce déréférence Data pour se rendre à l'ensemble des éléments Structure que vous pouvez attribuer à.

Answer 2

*Data[Number - 1] est équivalent à Data[Number - 1][0] parce que l'opérateur [] a la priorité plus élevée que unaire opérateur * et a[b] est équivalent à *((a) + (b)), de sorte *Data[Number - 1] = *(Data[Number - 1]) = *((Data[Number - 1]) + (0)) = [Data[Number - 1][0]

En outre, Data[1] ou au-dessus est hors de portée et il ne doit pas être accessible.

Je suppose que vous voulez (*Data)[Number - 1], ce qui équivaut à Data[0][Number - 1].