One case is when clone hits an assertion here, so the cleanupOnError gets ran before setMovePrimarySourceManager was called.

This can easily be fixed by either:
1. Make cleanup no-op if sourceMgr doesn't exist
or
2. Move the scopedGuard after uassert (less future proof)