diff options
| author | Clemens Buchacher <drizzd@aon.at> | 2009-05-27 22:13:43 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2009-05-27 23:16:16 -0700 |
| commit | 6a01554e6350123c78de805d820e90f1c56e5fdc (patch) | |
| tree | 512631158643b30cda638d18e1ca0ee70a7d3a8e /builtin-remote.c | |
| parent | e57cb0158209ece040a4b873064504efd7d2ec0e (diff) | |
| download | git-6a01554e6350123c78de805d820e90f1c56e5fdc.tar.gz git-6a01554e6350123c78de805d820e90f1c56e5fdc.tar.xz | |
fix segfault showing an empty remote
In case of an empty list, the search for its tail caused a
NULL-pointer dereference.
Signed-off-by: Clemens Buchacher <drizzd@aon.at>
Reported-by: Erik Faye-Lund <kusmabite@googlemail.com>
Acked-by: Jay Soffian <jaysoffian@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-remote.c')
| -rw-r--r-- | builtin-remote.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/builtin-remote.c b/builtin-remote.c index 71abf6840..fda9a54a0 100644 --- a/builtin-remote.c +++ b/builtin-remote.c @@ -299,11 +299,11 @@ static int get_push_ref_states(const struct ref *remote_refs, return 0; local_refs = get_local_heads(); - ref = push_map = copy_ref_list(remote_refs); - while (ref->next) - ref = ref->next; - push_tail = &ref->next; + push_map = copy_ref_list(remote_refs); + push_tail = &push_map; + while (*push_tail) + push_tail = &((*push_tail)->next); match_refs(local_refs, push_map, &push_tail, remote->push_refspec_nr, remote->push_refspec, MATCH_REFS_NONE); |