In connections_shutdown - check for Uninitialized struct first, since

In connections_shutdown - check for Uninitialized struct first, since
attempting to lock an unint'd mutex may crash on some platforms. And
there is no chance for an unint'd conn to get init'd while shutdown
is occurring, so the unprotected check of struct state is safe.