diff options
author | Jeff King <peff@peff.net> | 2021-04-28 11:42:30 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-04-29 10:03:46 +0900 |
commit | 1e951c647365be2e93bceddf7bd13173ed2cb1ce (patch) | |
tree | 51b69bcd12342bd618e41bc73afab7b06c1cf4d5 /pretty.c | |
parent | Git 2.31.1 (diff) | |
download | tgif-1e951c647365be2e93bceddf7bd13173ed2cb1ce.tar.xz |
pack-bitmap: clean up include_check after use
When a bitmap walk has to traverse (to fill in non-bitmapped objects),
we use rev_info's include_check mechanism to let us stop the traversal
early. But after setting the function and its data parameter, we never
clean it up. This means that if the rev_info is used for a subsequent
traversal without bitmaps, it will unexpectedly call into our
include_check function (worse, it will do so pointing to a now-defunct
stack variable in include_check_data, likely resulting in a segfault).
There's no code which does this now, but it's an accident waiting to
happen. Let's clean up after ourselves in the bitmap code.
Reported-by: David Emett <dave@sp4m.net>
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'pretty.c')
0 files changed, 0 insertions, 0 deletions