Uninitialized memory in EncodeTimeFromBeforeWindowsEpoch test case |
|||
Issue descriptionhttps://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%28DrMemory%20full%29%20%281%29/builds/5315 UNINITIALIZED READ: reading 0x002ef9f8-0x002ef9fa 2 byte(s) # 0 ntdll.dll!RtlTimeFieldsToTime +0x4a (0x77504ed4 <ntdll.dll+0x54ed4>) # 1 KERNELBASE.dll!SystemTimeToFileTime +0x4f (0x750c8730 <KERNELBASE.dll+0x8730>) # 2 base.dll!base::Time::FromExploded [base\time\time_win.cc:259] # 3 net::der::test::EncodeValuesTest_EncodeTimeFromBeforeWindowsEpoch_Test::TestBody [net\der\encode_values_unittest.cc:53] UNINITIALIZED READ: reading 0x003af8f8-0x003af8fa 2 byte(s) # 0 ntdll.dll!RtlTimeFieldsToTime +0x4a (0x77504ed4 <ntdll.dll+0x54ed4>) # 1 KERNELBASE.dll!SystemTimeToFileTime +0x4f (0x750c8730 <KERNELBASE.dll+0x8730>) # 2 base.dll!base::Time::FromExploded [base\time\time_win.cc:259] # 3 net::der::test::EncodeValuesTest_EncodeTimeAfterTimeTMax_Test::TestBody [net\der\encode_values_unittest.cc:79] # 4 testing::internal::HandleExceptionsInMethodIfSupported<> [testing\gtest\src\gtest.cc:2458]
,
Jul 15 2016
https://codereview.chromium.org/2148303002/ but that might not land today. I was out most of the day so the DrMemory bots error for this did not get suppressed.
,
Jul 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/20e1a9ebc85103a1a0d89461c9c2b268bbd5d6a2 commit 20e1a9ebc85103a1a0d89461c9c2b268bbd5d6a2 Author: thestig <thestig@chromium.org> Date: Fri Jul 15 02:50:57 2016 Make the DrMemory bots greener. - Disable failing chrome_elf BlacklistTests. - Suppress EncodeValuesTest uninit reads. - Update gamepad suppression after r403998. BUG= 371844 , 628177 , 628462 TBR=isheriff@chromium.org NOTRY=true Review-Url: https://codereview.chromium.org/2156473002 Cr-Commit-Position: refs/heads/master@{#405685} [modify] https://crrev.com/20e1a9ebc85103a1a0d89461c9c2b268bbd5d6a2/tools/valgrind/drmemory/suppressions.txt [modify] https://crrev.com/20e1a9ebc85103a1a0d89461c9c2b268bbd5d6a2/tools/valgrind/drmemory/suppressions_full.txt [modify] https://crrev.com/20e1a9ebc85103a1a0d89461c9c2b268bbd5d6a2/tools/valgrind/gtest_exclude/chrome_elf_unittests.gtest-drmemory_win32.txt
,
Jul 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/91a2817a25144df6fff7dde4dc04b951335aee0b commit 91a2817a25144df6fff7dde4dc04b951335aee0b Author: dadrian <dadrian@google.com> Date: Fri Jul 15 19:36:38 2016 Initialize memory in encode_values_unittest.cc BUG= 628177 Review-Url: https://codereview.chromium.org/2148303002 Cr-Commit-Position: refs/heads/master@{#405825} [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/net/cert/internal/parse_ocsp.h [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/net/cert/internal/parse_ocsp_unittest.cc [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/net/der/encode_values.cc [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/net/der/encode_values.h [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/net/der/encode_values_unittest.cc [modify] https://crrev.com/91a2817a25144df6fff7dde4dc04b951335aee0b/tools/valgrind/drmemory/suppressions.txt
,
Jul 20 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by dadrian@google.com
, Jul 14 2016