HEAP Corruption DETECTED错误该怎样解决？

vlinknight

LinkList ListDel(LinkList l){     LinkList p,q;     int i;     int s;         printf("\n请输入你要删除的位置：");         scanf("%d",&s);         while(s>ListSize)     {         printf("非法输入！请重新输入:\n");         scanf("%d",&s);     }     if(s==1)     {         p=l;         l=l->next;         free(p);//单步调试到这儿出的错         ListSize--;     }     else     {         q=l;         for(i=1;i<s-1;i++)         q=q->next;//用q保存第s个节点的前驱     p=q->next;     q->next=p->next;     free(p);     ListSize--;         }//end else     return l; } main里的调用是L=ListDel(L);

如果之前在第一个节点前插入过另一节点的话，就不会产生错误。否则显示Debug Error！HEAP Corruption DETECTED……

tyc611

没发现问题，可能是其它地方有问题吧

ilex

看标题是C++，看代码怎么又是C啊？

vlinknight

回一楼：哦，谢谢。现在直接把free（p）注释掉了，可以运行。
回二楼：编辑的时候没看到c分类，不好意思。

tyc611

原帖由 vlinknight 于 2007-10-27 19:24 发表
回一楼：哦，谢谢。现在直接把free（p）注释掉了，可以运行。
回二楼：编辑的时候没看到c分类，不好意思。

你这样会内存泄露

另外，刚发现你的程序在删除头结点时会出现错误，因为你并没有将修改后的头结点指针返回（应该使用LinkList *l传递参数才能将结果返回）

vlinknight

不太明白。原帖已经把l返回了，LinkList正是指针类型，main里是L=ListDel(L)，L是指针。
typedef struct Lnode{
        int data;
        struct Lnode *next;
}Lnode,*LinkList;

这是结点的定义，LinkList算是指针类型了吧？l本身就是指针，LinkList *l作参数的话是指针的指针？下面的next又应该怎么改啊？*l->next不行哦，谢谢。

vlinknight

而且十分奇怪的是当我在第一个结点前再插入一个结点的话（L值已改变）无论后来这个插入结点是否删除，上述错误并不会发生。难道是vc2005的一个bug？

tyc611

原帖由 vlinknight 于 2007-10-29 23:10 发表
不太明白。原帖已经把l返回了，LinkList正是指针类型，main里是L=ListDel(L)，L是指针。
typedef struct Lnode{
        int data;
        struct Lnode *next;
}Lnode,*LinkList;

这是结点的定义，LinkList算是指针类 ...

不好意思，没注意到你的返回值，程序是正确的
是不是你的传入参数有问题？或者你把完全程序拿上来帮你看看

vlinknight

我也不好意思。终于找到问题了，问题是L的第一次malloc初始化在main里，后面在函数体里free掉头结点好像会导致vc内存检查函数（误判？实际上可以忽略掉继续运行）。现在把main里的初始化改成函数形式了，结果完美通过了，不用注释掉free了。谢谢你的热心。