ich würd mal sagen ohne -ansi kompilieren. oder
char* line;
...
line = (char*) malloc(sizeof(char)*MAX_LENGTH+1);
Das ändern, was der Compiler bemängelt: Variablendeklarationen und Code sind vermischt. In ANSI C müssen alle Variablendeklarationen für einen Block am Beginn des Blocks erfolgen, erst dann darf eigentlicher Code (Funktionsaufrufe, Zuweisungen etc.) kommen.
Bei der Variante von ChristophM sollte man free() nicht vergessen.
Die Warnung bezieht sich auf die Reihenfolge von allen Deklarationen und allen sonstigen Statements in der gesamten Funktion. Entweder postest du also die ganze Funktion oder schaust selber drauf, daß zuerst nur Deklarationen kommen und dann nur Nichtdeklarationen.
Und Arrays kleiner konstanter Länge mit malloc anlegen ist total überflüssig. Total überflüssig ist übrigens auch der Cast (char *), der sollte weg.
