* [PATCH] Use correct default remote for fetching in submodule.
@ 2023-02-13 13:14 Mara
2023-02-13 13:45 ` Eric Sunshine
0 siblings, 1 reply; 2+ messages in thread
From: Mara @ 2023-02-13 13:14 UTC (permalink / raw)
To: git
"git submodule update" first tries a regular "git fetch"
to fetch the commit, but when that doesn't retrieve the commit
it wants, it tries "git fetch <remote> <commit>".
For <remote>, it used the wrong default remote: the default
remote of the outer repository, rather than the default remote
of the submodule.
Signed-off-by: Mara Bos <mara@marabos.nl>
---
builtin/submodule--helper.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index 4c173d8b37..50b96e0b9d 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -2225,7 +2225,10 @@ static int fetch_in_submodule(const char *module_path, int depth, int quiet,
strvec_pushf(&cp.args, "--depth=%d", depth);
if (oid) {
char *hex = oid_to_hex(oid);
- char *remote = get_default_remote();
+ char *remote;
+ int code = get_default_remote_submodule(module_path, &remote);
+ if (code)
+ return code;
strvec_pushl(&cp.args, remote, hex, NULL);
free(remote);
--
2.39.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Use correct default remote for fetching in submodule.
2023-02-13 13:14 [PATCH] Use correct default remote for fetching in submodule Mara
@ 2023-02-13 13:45 ` Eric Sunshine
0 siblings, 0 replies; 2+ messages in thread
From: Eric Sunshine @ 2023-02-13 13:45 UTC (permalink / raw)
To: Mara; +Cc: git
On Mon, Feb 13, 2023 at 8:31 AM Mara <mara@marabos.nl> wrote:
> "git submodule update" first tries a regular "git fetch"
> to fetch the commit, but when that doesn't retrieve the commit
> it wants, it tries "git fetch <remote> <commit>".
> For <remote>, it used the wrong default remote: the default
> remote of the outer repository, rather than the default remote
> of the submodule.
>
> Signed-off-by: Mara Bos <mara@marabos.nl>
> ---
I'm not a submodule user and I don't have any particular familiarity
with this code, so I may be wrong, but...
> diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
> index 4c173d8b37..50b96e0b9d 100644
> --- a/builtin/submodule--helper.c
> +++ b/builtin/submodule--helper.c
> @@ -2225,7 +2225,10 @@ static int fetch_in_submodule(const char *module_path, int depth, int quiet,
> - char *remote = get_default_remote();
> + char *remote;
> + int code = get_default_remote_submodule(module_path, &remote);
> + if (code)
> + return code;
> strvec_pushl(&cp.args, remote, hex, NULL);
> free(remote);
... it looks like this change may introduce a memory leak. Digging
down through get_default_remote_submodule() and the functions it
calls, it appears that repo_get_default_remote() can return a non-zero
code _after_ it has allocated memory for `remote`. If I'm reading this
correctly, then the above should probably be:
char *remote = NULL;
int code = get_default_remote_submodule(module_path, &remote);
if (code) {
free(remote);
return code;
}
Also, if possible, add a new test, perhaps to
t/t7406-submodule-update.sh, demonstrating that this change fixes the
problem and to ensure that it doesn't get broken again. If you have a
minimal-reproduction recipe which exhibits the problem, then you may
be able to turn it into an actual test.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-02-13 13:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-13 13:14 [PATCH] Use correct default remote for fetching in submodule Mara
2023-02-13 13:45 ` Eric Sunshine
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).