I looked at the patch in CVS. Seems like we should either also initialize idleError and tokenError in InitReq(), or make the check for these values in afs_Conn depend on initd == 1. As it stands, the test still relies on 2 potentially uninitialized variables, and servercheck=1 will have an effect or not depending on random values. Marc