Beh, per cominciare, per l'elemento

char year[4];

della struttura, devi prevedere 5 caratteri e non 4 per il terminatore ...

E poi ti consiglio di dare un'occhiata a questa linea

while (memb=false, memb=true);

perché non mi pare proprio corretta ...