summaryrefslogtreecommitdiff
path: root/tests/diff/diff_helpers.c
diff options
context:
space:
mode:
authorPatrick Steinhardt <ps@pks.im>2019-05-24 15:24:26 +0200
committerPatrick Steinhardt <ps@pks.im>2019-05-24 15:28:33 +0200
commitadd1743580149c3d1e570aafff3180cee216162e (patch)
treeee5c2b509779ff599365ed6bb0c7f605376e5037 /tests/diff/diff_helpers.c
parent2b8a05cc8ad1a9540ca690d1fe386123914e4922 (diff)
downloadlibgit2-add1743580149c3d1e570aafff3180cee216162e.tar.gz
cache: fix cache eviction using deallocated key
When evicting cache entries, we first retrieve the object that is to be evicted, delete the object and then finally delete the key from the cache. In case where the cache eviction caused us to free the cached object, though, its key will point to invalid memory now when trying to remove it from the cache map. On my system, this causes us to not properly remove the key from the map, as its memory has been overwritten already and thus the key lookup it will fail and we cannot delete it. Fix this by only decrementing the refcount of the evictee after we have removed it from our cache map. Add a test that caused a segfault previous to that change.
Diffstat (limited to 'tests/diff/diff_helpers.c')
0 files changed, 0 insertions, 0 deletions