초기화 Valgrind 오류

Question

이 함수는 해시 테이블의 일부입니다. 검색 할 해시 목록과 키를 전달하고 있습니다. 나는 그것을 실행하려고하면 불행하게도, Valgrind의 나에게 오류

==1741== Conditional jump or move depends on uninitialised value(s) 
==1741== at 0x4018DE: HashSearch (Hash.c:81) 
==1741== by 0x400FA7: function (Nine13.c:181) 
==1741== by 0x4009D8: main (Nine13.c:54) 
==1741== 
==1743== Conditional jump or move depends on uninitialised value(s) 
==1743== at 0x4018DE: HashSearch (Hash.c:81) 
==1743== by 0x400FA7: function (Nine13.c:181) 
==1743== by 0x4009D8: main (Nine13.c:54) 
==1743== 

주고 계속

int HashSearch (Hash_list h, char* key) 
{ 
    struct ent *x = (struct ent *)malloc(10*sizeof(struct ent)); 
    for (x = h->table[hash(key, h->size)]; x!=0; x=x->next) { 
     if (strcmp(x->pos, key)==0) { 
      return x->num; 
     } 
    } 
    return -1; 
} 

(라인 81 "에 대한"로 시작하는 코드의 라인입니다).

struct ent과 전달 된 해시 목록을 모두 초기화 한 것 같습니다. 무엇이 잘못 되었나요?

Answer 1

memset()을 사용하여 메모리를 지우거나 malloc() 대신 calloc() 기능을 사용하십시오.

Answer 2

우리 의견에 대한 귀하의 답변에서 귀하의 문제는 기능 밖에 있고 기능에서 사용 된 항목 중 하나가 제대로 초기화되지 않은 것 같습니다. 이 외부에서 코드의이 섹션에서 메모리 누수가 수행

struct ent *x = (struct ent *)malloc(10*sizeof(struct ent)); 
      ^^ 
    for (x = h->table[hash(key, h->size)]; x!=0; x=x->next) { 
     ^^^^^ 

당신이 x에 메모리를 할당 한 후 즉시 for 루프 x을 재 할당하는, 아마도 이것은 당신이해야 할 의미가 무엇인가 :

struct ent *x ; 
    for (x = h->table[hash(key, h->size)]; x!=0; x=x->next) { 

Answer 3

this other question에 따르면 h->table의 모든 항목이 초기화되지 않은 것이 문제였던 것으로 보입니다. table에 calloc 또는 memset을 사용하면 매우 높은 확률로 문제가 해결되지만 기술적으로는 이식 할 수 없습니다. 0은 널 포인터와 동일한 것으로 평가되지만, 비트 패턴은 모두 0이 될 수 있습니다. 또한 위의 다른 스레드와 내 주석에서 지적한 것처럼 복잡한 데이터 유형의 경우 다른 "빈"값을 원할 때가 많습니다.