summaryrefslogtreecommitdiff
path: root/diffcore-delta.c
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2012-10-28 07:40:00 -0400
committerLibravatar Jeff King <peff@peff.net>2012-10-28 07:59:44 -0400
commitb1c2f57db326c43ec286855b396fb16b4890cf2d (patch)
tree072f9b8a85cea0166e4fafd3df7cd1968d81dea7 /diffcore-delta.c
parentGit 1.7.12.4 (diff)
downloadtgif-b1c2f57db326c43ec286855b396fb16b4890cf2d.tar.xz
diff_grep: use textconv buffers for add/deleted files
If you use "-G" to grep a diff, we will apply a configured textconv filter to the data before generating the diff. However, if the diff is an addition or deletion, we do not bother running the diff at all, and just look for the token in the added (or removed) content. This works because we know that the diff must contain every line of content. However, while we used the textconv-derived buffers in the regular diff, we accidentally passed the original unmodified buffers to regexec when checking the added or removed content. This could lead to an incorrect answer. Worse, in some cases we might have a textconv buffer but no original buffer (e.g., if we pulled the textconv data from cache, or if we reused a working tree file when generating it). In that case, we could actually feed NULL to regexec and segfault. Reported-by: Peter Oberndorfer <kumbayo84@arcor.de> Signed-off-by: Jeff King <peff@peff.net>
Diffstat (limited to 'diffcore-delta.c')
0 files changed, 0 insertions, 0 deletions