diff options
| author | dkegel@google.com <dkegel@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-21 16:33:22 +0000 |
|---|---|---|
| committer | dkegel@google.com <dkegel@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-21 16:33:22 +0000 |
| commit | 62172f827c71701ecdad8b27999faf3074de7895 (patch) | |
| tree | b39e2de546f1f0a54eefd8ee4374d5ab2b0728ce /tools/valgrind/vbug205541.patch | |
| parent | 620d0a205bb8c59551afef4521a9c2bccedcf5c7 (diff) | |
| download | chromium_src-62172f827c71701ecdad8b27999faf3074de7895.zip chromium_src-62172f827c71701ecdad8b27999faf3074de7895.tar.gz chromium_src-62172f827c71701ecdad8b27999faf3074de7895.tar.bz2 | |
Update valgrind to fix one problem with 64 bit code, and a regression running wine.
BUG=valgrind_210481, valgrind_205541
TEST=valgrind net_unittests --gtest_filter=ProxyResolverV8Test.LoadLog ; valgrind wine's unit tests and see no HEAP_CreateSubHeap valgrind warnings
Review URL: http://codereview.chromium.org/306020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29655 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/valgrind/vbug205541.patch')
| -rw-r--r-- | tools/valgrind/vbug205541.patch | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/tools/valgrind/vbug205541.patch b/tools/valgrind/vbug205541.patch new file mode 100644 index 0000000..3ab721a --- /dev/null +++ b/tools/valgrind/vbug205541.patch @@ -0,0 +1,21 @@ +# See https://bugs.kde.org/show_bug.cgi?id=205541#c1 +Index: memcheck/mc_main.c +=================================================================== +--- memcheck/mc_main.c (revision 10877) ++++ memcheck/mc_main.c (working copy) +@@ -3708,7 +3708,14 @@ + if (rr || ww || xx) + MC_(make_mem_defined)(a, len); + else +- MC_(make_mem_noaccess)(a, len); ++ MC_(make_mem_defined)(a, len); ++ /* State after mmap(,,PROT_NONE,,,) should match ++ * state after mprotect(,,PROT_NONE): the operating system ++ * will trap any access, but any subsequent mprotect(,,PROT_R|W|X) ++ * will re-instate the previous V bits [in this case, all valid]. ++ * See comment in mc_pre_clo_init() re: ++ * VG_(track_change_mem_mprotect) ( NULL ); ++ */ + } + + static |