summaryrefslogtreecommitdiff
path: root/mergetools/gvimdiff3
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2018-11-20 04:48:57 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2018-11-21 13:50:27 +0900
commite159b8107190aa53b27f9f106e5874597106eb88 (patch)
tree17e142b9cd711961d8e719fc7db69a0c74426ca7 /mergetools/gvimdiff3
parentpack-objects: fix tree_depth and layer invariants (diff)
downloadtgif-e159b8107190aa53b27f9f106e5874597106eb88.tar.xz
pack-objects: zero-initialize tree_depth/layer arrays
Commit 108f530385 (pack-objects: move tree_depth into 'struct packing_data', 2018-08-16) started maintaining a tree_depth array that matches the "objects" array. We extend the array when: 1. The objects array is extended, in which case we use realloc to extend the tree_depth array. 2. A caller asks to store a tree_depth for object N, and this is the first such request; we create the array from scratch and store the value for N. In the latter case, though, we use regular xmalloc(), and the depth values for any objects besides N is undefined. This happens to not trigger a bug with the current code, but the reasons are quite subtle: - we never ask about the depth for any object with index i < N. This is because we store the depth immediately for all trees and blobs. So any such "i" must be a non-tree, and therefore we will never need to care about its depth (in fact, we really only care about the depth of trees). - there are no objects at this point with index i > N, because we always fill in the depth for a tree immediately after its object entry is created (we may still allocate uninitialized depth entries, but they'll be initialized by packlist_alloc() when it initializes the entry in the "objects" array). So it works, but only by chance. To be defensive, let's zero the array, which matches the "unset" values which would be handed out by oe_tree_depth() already. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'mergetools/gvimdiff3')
0 files changed, 0 insertions, 0 deletions