diff options
| author |  Jeff King <peff@peff.net> | 2016-09-12 12:46:17 -0400 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2016-09-12 12:09:41 -0700 |
| commit | 4b92bae7d363adfe923337f198f9c98dc1c68e27 (patch) | |
| tree | e6ba67e050ccd65e8baf5ffcfa6b08c0ee7c4bbb /builtin/pack-objects.c | |
| parent | t/perf: add basic perf tests for delta base cache (diff) | |
| download | tgif-4b92bae7d363adfe923337f198f9c98dc1c68e27.tar.xz | |
add_delta_base_cache: use list_for_each_safe
We may remove elements from the list while we are iterating,
which requires using a second temporary pointer. Otherwise
stepping to the next element of the list might involve
looking at freed memory (which generally works in practice,
as we _just_ freed it, but of course is wrong to rely on;
valgrind notices it).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/pack-objects.c')
0 files changed, 0 insertions, 0 deletions