summaryrefslogtreecommitdiff
path: root/replace-object.h
diff options
context:
space:
mode:
authorMatheus Tavares <matheus.bernardino@usp.br>2020-01-15 23:39:51 -0300
committerJunio C Hamano <gitster@pobox.com>2020-01-17 13:52:14 -0800
commitd5b0bac52841857b5ba197ca931ecf729fdbc63e (patch)
treec62f4c933d0c0f79cde157c53c081308f624f2d9 /replace-object.h
parentfaf123c730df1a545e2dc7ca82fd437e93fc519c (diff)
grep: fix racy calls in grep_objects()
deref_tag() calls is_promisor_object() and parse_object(), both of which perform lazy initializations and other thread-unsafe operations. If it was only called by grep_objects() this wouldn't be a problem as the latter is only executed by the main thread. However, deref_tag() is also present in read_object_file()'s call stack. So calling deref_tag() in grep_objects() without acquiring the grep_read_mutex may incur in a race condition with object reading operations (such as the ones internally performed by fill_textconv(), called at fill_textconv_grep()). The same problem happens with the call to gitmodules_config_oid() which also has parse_object() in its call stack. Fix that protecting both calls with the said grep_read_mutex. Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'replace-object.h')
0 files changed, 0 insertions, 0 deletions