C言語 構造体配列のコピーが上手くいきません

Question

前提・実現したいこと

C言語の練習をしています。
構造体配列とハッシュ関数の勉強のために以下のコードを書きました。
あらかじめ用意したコピー元の構造体配列(メンバは名前と電話番号)を、名前から計算したハッシュ値をもとにコピー先の構造体ポインタ配列に格納したいです。しかし、以下の通り上手くいっておりません。色々悩みましたが、いまだコードのどの部分が悪いかの分かっておりません。

発生している問題

予め用意した構造体配列(コピー元)をハッシュ関数で求めたハッシュ値をもとに構造体ポインタ配列(コピー先)にfor文で一件づつ格納しているつもりが、コピー終了後の構造体ポインタ配列のデータは、全て最後にコピーしたデータ(ここでは"Ogawa")になってしまっている。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define SIZE 71
#define NAME_MAX 16
#define NUMBER_MAX 16

struct TelephoneBook{
	char name[NAME_MAX];
	char number[NUMBER_MAX];
};

//ハッシュ値を計算して返す関数
unsigned int hash(char *s){
	unsigned int ret = 0;

	while(*s != '\0'){
		ret = ret * 37 + *s;
		s++;
	}
	return ret % SIZE;
}

int main(void){
	//コピー元　構造体配列
	struct TelephoneBook table_src[SIZE] = {
		{"Sato",		"090-1234-XXXX"},
		{"Suzuki", 		"080-1234-YYYY"},
		{"Tanaka", 		"080-1234-ZZZZ"},
		{"Yamada",		"090-1111-2222"},
		{"Takahashi",	"080-2222-3333"},
		{"Ogawa",		"080-3333-4444"}
	};

	//コピー先　構造体のポインタ配列
	struct TelephoneBook *table_dst[SIZE] = {NULL};

	for(int i = 0; table_src[i].name[0] != '\0'; i++){
		struct TelephoneBook temp;

		strcpy(temp.name, table_src[i].name);
		strcpy(temp.number, table_src[i].number);

		//ハッシュ値の計算
		int hashval = hash(table_src[i].name);

		//ハッシュ値の計算結果をもとに、コピー先のテーブルにtempのアドレスを格納
		table_dst[hashval] = &temp;

		//testprint
		printf("table_dst[%d]->name is %s\n", hashval, table_dst[hashval]->name);
	}

	//testprint
	puts("");
	for(int i = 0; i < SIZE; i++){
		if(table_dst[i] != NULL){
			printf("table_dst[%d]->name is %s\n", i, table_dst[i]->name);			
		}
	}

	return EXIT_SUCCESS;
}

試したこと

プログラムにテストプリントを入れてfor文中のコピー結果と、for文終了後のコピー結果を確認しました。for文中は正しくコピーされているように見えますが、for文終了後はすべて最後にコピーした"Ogawa"になってしまいます。

補足情報（FW/ツールのバージョンなど）

gccでコンパイルして実行ファイルを作っています。

以上、宜しくお願いします。

Answer 1

for(...) {
struct TelephoneBook temp;
...
//ハッシュ値の計算結果をもとに、コピー先のテーブルにtempのアドレスを格納
table_dst[hashval] = &temp;
}

tempの生存期間はfor文の、しかもその一周だけです。
tempが死んでいるのにも関わらず、アドレスから不正アクセスしようとしているのが問題です。

これを避けるためには、mallocで領域を確保してあげれば良いでしょう。
ただしメモリリークには注意してください。

Comments

[Tteratail]

変数の生存期間でしたか。
mallocで確保すれば消えないんですね。そしてそのあとfreeを忘れないようにします。

Answer 2

つtable_dst[hashval] = &table_src[i];

tempの値はループ毎に再確保され、ループを抜けると破棄されます。