J’obtiens une erreur de segmentation (core dumped) lorsque j’ai essayé d’exécuter le programme avec struct

Je dois lire un fichier et stocker les données du fichier dans une structure. La première ligne du fichier contient la taille du tableau de structures que je dois allouer dynamicment.

4 12/04/2010 Interview went well I think, though was told to wear shoes. 18/04/2010 Doc advised me to concentrate on something... I forget. 03/05/2010 Was asked today if I was an art exhibit. 19/05/2010 Apparently mudcakes not made of mud, or angry wasps.

Je dois exécuter mon code parfaitement sous Windows, mais lorsque je cours dans un environnement Unix, cela me montre une erreur de segmentation (core dumped). J’ai utilisé valgrind pour vérifier la fuite de mémoire et c’est le résultat

  ==4344== Invalid read of size 1 ==4344== at 0x407F842: ____strtol_l_internal (strtol_l.c:298) ==4344== by 0x407F606: strtol (strtol.c:108) ==4344== by 0x407C87E: atoi (atoi.c:27) ==4344== by 0x8048837: main (in /home/admininistrator/ucp/p6/gg) ==4344== Address 0x0 is not stack'd, malloc'd or (recently) free'd ==4344== ==4344== ==4344== Process terminating with default action of signal 11 (SIGSEGV) ==4344== Access not within mapped region at address 0x0 ==4344== at 0x407F842: ____strtol_l_internal (strtol_l.c:298) ==4344== by 0x407F606: strtol (strtol.c:108) ==4344== by 0x407C87E: atoi (atoi.c:27) ==4344== by 0x8048837: main (in /home/admininistrator/ucp/p6/gg) ==4344== If you believe this happened as a result of a stack ==4344== overflow in your program's main thread (unlikely but ==4344== possible), you can try to increase the size of the ==4344== main thread stack using the --main-stacksize= flag. ==4344== The main thread stack size used in this run was 8388608. ==4344== ==4344== HEAP SUMMARY: ==4344== in use at exit: 1,396 bytes in 3 blocks ==4344== total heap usage: 3 allocs, 0 frees, 1,396 bytes allocated ==4344== ==4344== LEAK SUMMARY: ==4344== definitely lost: 0 bytes in 0 blocks ==4344== indirectly lost: 0 bytes in 0 blocks ==4344== possibly lost: 0 bytes in 0 blocks ==4344== still reachable: 1,396 bytes in 3 blocks ==4344== suppressed: 0 bytes in 0 blocks ==4344== Rerun with --leak-check=full to see details of leaked memory ==4344== ==4344== For counts of detected and suppressed errors, rerun with: -v ==4344== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0) Segmentation fault (core dumped)

Voici mon code ci-joint

 #include #include #include #include"struct.h" int main(int argc, char* argv[]) { if (argc  2) { printf("You have enter too many arguments."); } else { FILE *file; Diary *res; Diary *res2; char line[102]; int i, size, k, l, choice; int day, month, year; /* int d[10],m[10],y[10];*/ char as[102]; char* oken; char* yoken; char* coken; oken = NULL; yoken = NULL; coken = NULL; i = 0; file = fopen("struct.txt", "r"); if (file == NULL) { perror("Error opening file\n."); } else { fscanf(file, "%d", &size); res = (Diary*) malloc(size * sizeof(Diary)); res2 = (Diary*) calloc((5), sizeof(Diary)); while (fgets(line, sizeof(line), file) != NULL) { oken = strtok(line, "/"); if (oken != NULL) { res2[i].day= atoi(oken); coken = strtok(NULL, "/"); if (oken != NULL) { res2[i].month = atoi(coken); yoken = strtok(NULL, "\n "); if (coken != NULL) { /*printf("%s",yoken);*/ res2[i].year = atoi(yoken); fgets(as, 102, file); strncpy(res2[i].entry, as, 102); } } } i++; } k = 1; l = 0; while (l < size) { res[l].day = res2[k].day; res[l].month = res2[k].month; res[l].year = res2[k].year; strncpy(res[l].entry, res2[k].entry, 102); k++; l++; } choice = atoi(argv[1]); printf("%d-%02d-%02d:%s",res[choice].year, res[choice].month,res[choice].day,res[choice].entry); free(res2); free(res); } fclose(file); } return 0; }

J’ai besoin de lire toutes les données du fichier dans la structure et de l’imprimer chaque fois que l’utilisateur souhaite cette entrée. J’ai essayé de déboguer une partie à la fois, et j’ai découvert que c’était la partie while( fgets( line, sizeof( line ), file) != NULL) qui donne le problème. Mais je n’ai aucune idée de comment le réparer.

Mon struct.h est donné comme ci-dessous:

  typedef struct journal{ int day; int month; int year; char entry[1024]; } Diary;

Je ne comprends pas très bien tout ce que vous essayez de réaliser, mais voici quelques problèmes.

Vous n’avez probablement pas besoin des variables de day month year .
Cette ligne

day = atoi(oken);

devrait probablement être

res2[i].day = atoi(oken);
Il y a un problème avec la ligne qui lit la size

fscanf(file, "%d", &size)

Ceci lit un entier mais il ne lit pas la nouvelle ligne

Vous devez changer cela pour être quelque chose comme

fscanf(file, "%d\n", &size)

ou utiliser des fgets .

En raison de la nouvelle ligne de fin, la prochaine fois que vous appelez, vous obtenez une chaîne contenant uniquement la nouvelle ligne.
Vos appels strtok et contrôles NULL sont déphasés. Le premier, pour oken , oken va. Mais ensuite, vous faites un retour coken mais une vérification NULL sur oken et enfin un strtok yoken retour et une vérification NULL sur coken . Dans les 3 cas, l’appel à strtok doit être suivi d’une vérification NULL sur la valeur renvoyée (comme c’est le cas pour oken ).
Je ne comprends pas le but de la boucle while (l < size) (peut-être à cause de la mauvaise manipulation de la nouvelle ligne comme décrit ci-dessus?). Vous allouez 5 structs, lisez 4 de structs.txt dans res2 (éléments 0 à 3) puis vous copiez les éléments 1 à 4 de res2 dans les éléments 0 à 3 de res . Cela signifie que l'élément 0 de res2 n'est pas copié et que l'élément 4, qui ne contient que des zéros, est copié.

La cause de l'accident est une combinaison des points 3 et 4.

Je déconseille d'utiliser atoi car il ne fait pas de vérification d'erreur. Il est dangereux d'utiliser sauf si vous êtes certain que la chaîne contient un entier bien formé. Si vous voulez que votre code soit robuste, vous devez append plus de vérification des erreurs, par exemple, vérifiez les valeurs de retour de malloc et de calloc .