Re: [PATCH v2 7/8] submodule-config: pass repo upon blob config read

From: Jonathan Tan <jonathantanmy@google.com>
Date: Aug 16, 2021, 20:09 UTC
Message-ID: <20210816200938.1639871-1-jonathantanmy@google.com>
In-Reply-To: <CAHd-oW4rd6=9WP-7k-Z9V1Z-4nDXR13-45cjyB6GEczxD3+56g@mail.gmail.com>

Show 5 quoted lines

> > @@ -1827,7 +1829,7 @@ static int git_config_from_blob_ref(config_fn_t fn,
> >
> >         if (get_oid(name, &oid) < 0)
> 
> This should be `repo_get_oid(repo, ...)` now.

Ah, good catch! This wasn't caught by the tests because the submodule config mechanism always passes a full-length hexadecimal string hash as "name" - and probably would never be caught because git_config_from_blob_ref() is only called from config_with_options(), which is called with non-NULL source from only 2 files: submodule-config.c (this one) and builtin/config.c (which most likely will never operate on any repo other than the_repository). I'll refactor the API to avoid this situation in the first place.