我是valgrind的新手,我用一些我为四叉树写的代码来运行它。
我编写了一个函数,以递归方式从四叉树中释放节点:
void destroyTree (Node* p)
{
if (!p) return;
for (size_t i = 0; i < 4; ++i)
destroyTree(p->child[i]);
free(p);
}
我在main函数中调用该函数:
int main( int argc, char **argv ) {
Node *head;
// make the head node
head = makeNode( 0.0,0.0, 0 );
// make a tree
makeChildren( head );
makeChildren( head->child[0] );
makeChildren( head->child[1] );
makeChildren( head->child[2] );
makeChildren( head->child[3] );
// print the tree for Gnuplot
writeTree( head );
return 0;
//destroy tree
destroyTree (head);
return 0;
}
当我运行valgrind时,它表明我失去了一些记忆。
结构是:
struct qnode {
int level;
double xy[2];
struct qnode *child[4];
};
typedef struct qnode Node;
我在buildTree中调用malloc:
Node *makeNode( double x, double y, int level ) {
int i;
Node *node = (Node *)malloc(sizeof(Node));
node->level = level;
node->xy[0] = x;
node->xy[1] = y;
for( i=0;i<4;++i )
node->child[i] = NULL;
return node;
}
我该如何阻止泄漏?我的解放功能有问题还是在其他地方?
好吧,因为没有人发现这一点,qazxsw poi的结尾写道:
main
我们注意到最后两行是不可缓存的,因为在此之上有一个 // print the tree for Gnuplot
writeTree( head );
return 0; // <----------- wooooot!
//destroy tree
destroyTree (head);
return 0;
}
语句。所以从不调用return 0
方法,这解释了内存泄漏。
始终启用编译器警告并阅读它们。控制流程很简单,每个编译器都在这里检测死代码。
也就是说,即使destroyTree
在这里不起作用,必须明确添加-Wall -Wextra
标志(-Wunreachable-code
)