diff options
| author | Jeff King <peff@peff.net> | 2022-09-08 19:24:21 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-09-08 20:10:32 (GMT) |
| commit | 080bc4990f56f9d0687919e5136cb138509f1269 (patch) | |
| tree | 34e8c88a9d129b2d51bee46cc58e22ad4148c0fd /builtin/fetch.c | |
| parent | ac8035a2affdf30f2c691ad760826d955bba0507 (diff) | |
| download | git-080bc4990f56f9d0687919e5136cb138509f1269.zip git-080bc4990f56f9d0687919e5136cb138509f1269.tar.gz git-080bc4990f56f9d0687919e5136cb138509f1269.tar.bz2 | |
fetch: stop checking for NULL transport->remote in do_fetch()
This field will never be NULL; if it were, we'd segfault earlier in the
function when we unconditionally check transport->remote->fetch_tags.
Likewise, many other functions dereference it unconditionally.
This is a small simplification, but it will make things easier as we
extend this conditional in the next patch.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/fetch.c')
| -rw-r--r-- | builtin/fetch.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/fetch.c b/builtin/fetch.c index ac29c2b..d39098d 100644 --- a/builtin/fetch.c +++ b/builtin/fetch.c @@ -1626,7 +1626,7 @@ static int do_fetch(struct transport *transport, break; } } - } else if (transport->remote && transport->remote->fetch.nr) + } else if (transport->remote->fetch.nr) refspec_ref_prefixes(&transport->remote->fetch, &transport_ls_refs_options.ref_prefixes); |