Should we be more defensive here? `strchr` may return a null pointer.