null check for server missing - you rely upon it not being null later on. Also, formatting - surround = with spaces