git: do not validate submodules of fresh checkouts

Fixes #14603

Author: osiewicz

src/cargo/sources/git/utils.rs

@@ -194,19 +194,20 @@ impl GitDatabase {
         // A non-fresh checkout can happen if the checkout operation was
         // interrupted. In that case, the checkout gets deleted and a new
         // clone is created.
-        let (checkout, guard) = match git2::Repository::open(dest)
+        let checkout = match git2::Repository::open(dest)
             .ok()
             .map(|repo| GitCheckout::new(self, rev, repo))
             .filter(|co| co.is_fresh())
         {
-            Some(co) => (co, None),
-            None => GitCheckout::clone_into(dest, self, rev, gctx)
-                .map(|(co, guard)| (co, Some(guard)))?,
+            Some(co) => co,
+            None => {
+                let (checkout, guard) = GitCheckout::clone_into(dest, self, rev, gctx)?;
+                checkout.update_submodules(gctx)?;
+                guard.mark_ok()?;
+                checkout
+            }
         };
-        checkout.update_submodules(gctx)?;
-        if let Some(guard) = guard {
-            guard.mark_ok()?;
-        }
+
         Ok(checkout)
     }