I don't think User* is ever NULL when passed to things like this so these checks are probably not needed.