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)); 

您只为一个结构分配空间。紧接着您读取多个结构,从而覆盖内存。

于 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 回答