Revert "Return something from read_externals() to keep gcc happy."

read_externals can't ever return NULL, since err doesn't return. gcc's warning
is wrong. We should think about annotating the function to make that clear
rather than putting in a check that can never trigger.