Hmm. Unless something else depends on e_type being ET_NONE or something, it seems like it would be wiser just to set it earlier. The assert is pretty sensible in that something is very wrong if we get there for a non-ET_REL file.