不经意间的内存泄露

今天在做HDOJ上的1800 题，题目描述的很复杂，其实解题思路归纳起来就这么一句话：

　　“给你一系列数字，找出出现次数最多的那个”

由于测试样本给的数字可能很大，范围超出__int64的表示范围，所以只能用个字符串来写大数了

用的是C语言，也只能自己写hash算法。

由于是多组测试数据，所以每次都要把hash表清空，开始随手写了个

for(i=0;i<NHASH;i++)
      carr[i] = NULL;

代码提交后，不管怎么样，AC了，不过内存使用虽然在AC范围内，但也出奇的大。

想了半天，是上面代码的问题，因为数组里面每个元素都可能是长度大于一的链表，结构如下：

typedef struct MyCount MyCount;
struct MyCount {
    char num[32];
    int cnt;
    MyCount *next;
};

所以很显然，我几乎没有正确的free已经分配出去的内存

而free链表的话，很容易就会引起内存泄露，正确的free链表的代码如下（来自《Expert C Programming》）：

struct node *p, *start, *tmp;
for(p=start; p; p=tmp){
    tmp = p->next;
    free(p);
}

另附上AC代码，AC的效率不高，但是也算是可作为参考用C实现了一个简单的hash表

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MULTIPLIER 31
#define NHASH 1023
typedef struct MyCount MyCount;
struct MyCount {
    char num[32];
    int cnt;
    MyCount *next;
};
MyCount *carr[NHASH];
int hash(char *str) {
    unsigned int h;
    unsigned char *p;
    h=0;
    for(p=(unsigned char *)str; *p!='\0'; p++)
        h = MULTIPLIER * h + *p;
    return h%NHASH;
}
void add(char *str) {
    int h;
    int findit = 0;
    MyCount *sp;
    h = hash(str);
    for(sp = carr[h]; sp!=NULL; sp = sp->next) {
        if(strcmp(str,sp->num) == 0){//find it
            sp->cnt++;
            findit = 1;
            break;
        }
    }
    if(findit==0){//did not find it
        sp = (MyCount *)malloc(sizeof(MyCount));
        strcpy(sp->num,str);
        sp->cnt = 1;
        sp->next = carr[h];
        carr[h] = sp;
    }
}


int main() {
    int n;
    char temp[32];
    char *p;
    int i,max;
    MyCount *sp;
    while(scanf("%d",&n)!=EOF) {
        for(i=0;i<NHASH;i++)
            carr[i] = NULL;
        max = 0;
        while(n--) {
            scanf("%s",temp);
            p = temp;
            while(*p=='0')
                p++;
            add(p);
        }
        for(i=0;i<NHASH;i++){
            sp = carr[i];
            while(sp){
                if(sp->cnt>max)
                    max = sp->cnt;
                //printf("##%s %d\n",sp->num,sp->cnt);
                sp = sp->next;
            }
        }
        printf("%d\n",max);
    }
    return 0;
}