From: Julian Seward Date: Mon, 28 Aug 2006 22:59:58 +0000 (+0000) Subject: Merge r6040 (fix for: 'cfsi->len > 0 && cfsi->len < 2000000' failed) X-Git-Tag: svn/VALGRIND_3_2_1~15 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7c6c07dd72c8376ef0182ab0a8c270247d375ee5;p=thirdparty%2Fvalgrind.git Merge r6040 (fix for: 'cfsi->len > 0 && cfsi->len < 2000000' failed) git-svn-id: svn://svn.valgrind.org/valgrind/branches/VALGRIND_3_2_BRANCH@6041 --- diff --git a/coregrind/m_debuginfo/storage.c b/coregrind/m_debuginfo/storage.c index 066cb920ea..2f103a9036 100644 --- a/coregrind/m_debuginfo/storage.c +++ b/coregrind/m_debuginfo/storage.c @@ -308,7 +308,13 @@ void ML_(addDiCfSI) ( struct _SegInfo* si, DiCfSI* cfsi ) ML_(ppDiCfSI)(cfsi); } - vg_assert(cfsi->len > 0 && cfsi->len < 2000000); + /* sanity */ + vg_assert(cfsi->len > 0); + /* If this fails, the implication is you have a single procedure + with more than 5 million bytes of code. Which is pretty + unlikely. Either that, or the debuginfo reader is somehow + broken. */ + vg_assert(cfsi->len < 5000000); /* Rule out ones which are completely outside the segment. These probably indicate some kind of bug, but for the meantime ignore