携帯からだからテンプレに沿ってなくてすまん

簡単なデータベースマネジメントシステムをCで作ってるんだがデータをロードする際の
while (fgets(buf, 100, fp) != NULL) {
strcpy(p->name, buf);
}
これは何がいけないんだろう
pは構造体を指すポインタでnameはその中のchar型の配列
bufもchar型の配列