この関数を呼び出す時の第一引数に当てる変数は
どのように定義されてますか?

int main()
{
 int　**img;//この変数定義で

 load_ppm_file(img,(後は省略));//第一引数に持ってくるとNG

}

上記のようなことやってませんか?

一次元配列なの？多分みんな二次元だと思ってるような。
いや、それだとポインタのポインタのポインタが必要か。

とりあえず、一次元ならこうかな。

*img =(int*)malloc(sizeof(int)*(*width) * (*height) * 3);
if(*img==NULL){
    printf("ERROR");
    exit(1);
} 
for (x = 0; x < (*width) * (*height) * 3; x++){
    (*img)[x] =( 0x00ff & fgetc(fp)); 
}

heightとwidthをポインタで渡す必要はないよね。
あと関数の最後でfree(*img);してしまってるけどいいのでしょうか？
関数内限定ならimgを引数で渡す必要さえないのですが。

個人的には、確保した配列の先頭アドレスを返すようにすれば、
余計なポインタのポインタとか考えなくてすむので楽だと思います。

あとお約束の一言（重要）
ソースを添付するときは「HTML変換ツール」を使ってください。

int load_ppm_file(int *img, const char *filename, int *width, int *height)
{
    // ...snip

    img =(int*)malloc(sizeof(int) * (*width) * (*height) * 3);

    if(img==NULL){
        printf("ERROR");
        exit(1);
    }

    for (x = 0; x < (*width) * (*height) * 3; x++){
        img[x] =( 0x00ff & fgetc(fp));
    }

    fclose(fp);
    free(img);
    return 0;
}