]> git.ipfire.org Git - thirdparty/valgrind.git/commit
Always check crc in open_debug_file for now. Bug #338791 followup.
authorMark Wielaard <mark@klomp.org>
Fri, 5 Sep 2014 22:50:51 +0000 (22:50 +0000)
committerMark Wielaard <mark@klomp.org>
Fri, 5 Sep 2014 22:50:51 +0000 (22:50 +0000)
commit13e43db4830e9673db6cd4e5dbf9684c10720c73
tree9dd66d39035742cd9615ebac3b5bb736ce573bdd
parent50618eb0247326346cf7107dabf91a327cac8eac
Always check crc in open_debug_file for now. Bug #338791 followup.

Revision r14464 made it so that debug alt files could be found by their
build-id or their (relative) file path. Debug alt files are matched using
the given build-id, but by crc. Calculating the full CRC is costly, but
currently still needed to avoid misidentifying the main file as debug
file. Slightly more efficient would be to use fstat to check we aren't
actually opening the main file under any other name (but that only works
for local DiImages). Or we could check that the file being opened actually
has at least one .debug* section. But this change was the minimal patch
to make things work as before.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@14474
coregrind/m_debuginfo/readelf.c