1

プログラムを実行すると、次のようなエラーが発生します。

*** glibc detected *** ./prog: double free or corruption (!prev): 0x09155170 ***

問題は明らかにmallocを介して構造体にメモリを与えることですが、何が問題なのかわかりません。これが私のプログラムのコードです(Cで):

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

typedef struct 
{
      char kood[4];
      int kogus;
      char nimetus[80];
      double hind;
      int P;
      int K;
      int A;
}ese;

void sort(int m, ese* d);

void search_kood(int m, ese* d);

void search_nimetus(int m, ese* d);

void search_kuupaev(int m, ese* d);

int menu();

int main (void)
{
    FILE *list;
    list = fopen("elektroonikapood.txt", "r");
    int menu_valik,m,i;
    m=0;
    if (list==NULL)
      {
            printf("Empty or corrupted file!");
            getchar();
            return 0;   
      }  
    ese *esemed = (ese*) malloc(sizeof(ese)); 
    while(!feof(list)) 
      {

                  fscanf(list, "%s", esemed[m].kood);
                  fscanf(list, "%d", &esemed[m].kogus);
                  fscanf(list, "%s", esemed[m].nimetus);
                  fscanf(list, "%lg", &esemed[m].hind);
                  fscanf(list, "%d", &esemed[m].P);
                  fscanf(list, "%d", &esemed[m].K);
                  fscanf(list, "%d", &esemed[m].A);
                  m++;
      }


    while(1)
   { menu_valik = menu();
    if(menu_valik == 1)
      sort(m, esemed);
    else if (menu_valik == 2)
      search_kood(m, esemed);
    else if (menu_valik == 3)
      search_kuupaev(m, esemed);
    else if (menu_valik == 4)
      search_nimetus(m, esemed);
    else if (menu_valik == 0)
        {
          free(esemed);
          fclose(list);
          exit(1);
       }
    else
      break;
     }
     return 0;
}

追加機能もありますが、特に問題はないと思います。

4

2 に答える 2

4

この声明では：

ese *esemed = (ese*) malloc(sizeof(ese));

1 つの構造にのみスペースを割り当てます。直後に複数の構造体を読み込み、それによってメモリを上書きします。

于 2013-04-05T13:07:57.707 に答える
1

まず... C で malloc をキャストしないでください。エラーを隠すことができます。

第二に...あなたのmallocはループの外にありwhile( !feof( list ) )ますが、それはese構造の単一のインスタンスしか割り当てませんが、ループは複数のインスタンスをロードしていることを意味します...つまり、メモリを上書きしています...その時点ですべての賭けはオフです。

   ese *esemed = NULL; 
   ese *lastItem = NULL;

   while(!feof(list)) 
   {
        ese* newItem = malloc( sizeof( ese ) );

        if( !esemed ) esemed = newItem;
        if( lastItem ) lastItem->next = newItem;

        lastItem = newItem;

        fscanf(list, "%s", newItem->kood);
        fscanf(list, "%d", &newItem->kogus);
        fscanf(list, "%s", newItem->nimetus);
        fscanf(list, "%lg", &newItem->hind);
        fscanf(list, "%d", &newItem->P);
        fscanf(list, "%d", &newItem->K);
        fscanf(list, "%d", &newItem->A);
    }
于 2013-04-05T13:09:08.633 に答える