前提・実現したいこと

Cで構造体を利用してbaseball_player.defというファイルを読み込み、各データを逆順に表示するプログラムなのですが、コアダンプになってしまいます。読み込んだデータを格納するポインタ配列data[MAX_LEN]を二次元配列data[player_num][MAX_LEN]にしてみたりもしましたがうまくいきませんでした。
どこが間違っているのでしょうか？できれば詳しい解説等もつけてくださればありがたいです。
変更するのはmain関数内のみで、関数等は変更しないようにお願いします。

ソースコード

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

#define MAX_LEN 100

/*PLAYER構造体の定義*/
typedef struct string
{
    char name[10]; /*名前*/
    int number;  /*背番号*/
    int at_bat;  /*打数*/
    int hit;     /*安打数*/
    double ave;　/*打率*/
}PLAYER;



/*makePlayerData関数の定義*/
PLAYER makePlayerData(char *data)
{
    PLAYER a;

    /*文字列より各種データの読み込み*/
    sscanf(data, "%s %d %d %d", a.name, &a.number, &a.at_bat, &a.hit);
    /*打率の計算*/
    a.ave = (double)a.hit / (double)a.at_bat;

    return a;
}



/*printPlayerData関数の定義*/
void printPlayerData(PLAYER a){
    printf("%s     %d   %d   %d   %f\n", a.name, a.number, a.at_bat, a.hit, a.ave);
}



int main(int argc, char *argv[])
{
    char buff[MAX_LEN];
    char *filename;
    FILE *fp;
    int i;
    int player_num;
    char *data[MAX_LEN];
    PLAYER p[player_num];


    if(argv[1] != NULL) filename = argv[1];

    /*コマンドラインパラメータの数が間違っている場合の処理*/
    if(argc != 2){
        printf("Usage: ./print_player_data.c [filename]\n");
        exit(1);
    }

    /*ファイルを開く*/
    fp = fopen(filename, "r");

    /*ファイルが存在しない場合の処理*/
    if(fp == NULL) {
        printf("File %s doesn't exist．\n", filename);
        exit(1);
    }

    /*ファイルの読み込み*/
    i = 0;
    while(fgets(buff, sizeof(buff) , fp) != NULL){
        sscanf(buff, "%s", data[i]);
        p[i] = makePlayerData(data[i]);
        i++;
    }

    /*ファイルを閉じる*/
    fclose(fp);

    /*データの表示*/
    player_num = i;
    for(i = player_num - 1; i >= 0; i--){
        printPlayerData(p[i]);
    }

    return 0;
}

baseball_player.defの内容

Tanaka 3 483 125
Yamada 9 509 143
Takahashi 24 503 152
Abe 24 498 138
Suzuki 31 518 149