From 7ba80c728420735950f30f43095e74e9b2d294e0 Mon Sep 17 00:00:00 2001 From: Tobias Sargeant Date: Tue, 7 Feb 2017 14:33:48 +0000 Subject: Address post-submit review comments related to CL #430050 See: https://chromium-review.googlesource.com/c/430050/ BUG=664460 Change-Id: I3cbfbd5b00725bd501f06427eebd976267c4f617 Reviewed-on: https://chromium-review.googlesource.com/438444 Reviewed-by: Primiano Tucci --- src/client/linux/minidump_writer/minidump_writer_unittest.cc | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'src/client/linux/minidump_writer/minidump_writer_unittest.cc') diff --git a/src/client/linux/minidump_writer/minidump_writer_unittest.cc b/src/client/linux/minidump_writer/minidump_writer_unittest.cc index b9d8cd0d..f3b78a22 100644 --- a/src/client/linux/minidump_writer/minidump_writer_unittest.cc +++ b/src/client/linux/minidump_writer/minidump_writer_unittest.cc @@ -241,7 +241,8 @@ TEST(MinidumpWriterTest, StacksSkippedIfRequested) { // pass an invalid principal mapping address, which will force // WriteMinidump to not dump any thread stacks. ASSERT_TRUE(WriteMinidump(templ.c_str(), child, &context, sizeof(context), - true, 0x0102030405060708, false)); + true, static_cast(0x0102030405060708ull), + false)); // Read the minidump. And ensure that no thread memory was dumped. Minidump minidump(templ); @@ -286,7 +287,12 @@ TEST(MinidumpWriterTest, StacksAreSanitizedIfRequested) { Minidump minidump(templ); ASSERT_TRUE(minidump.Read()); - const uintptr_t defaced = 0X0DEFACED0DEFACEDull; + const uintptr_t defaced = +#if defined(__LP64__) + 0x0defaced0defaced; +#else + 0x0defaced; +#endif MinidumpThreadList *threads = minidump.GetThreadList(); for (unsigned int i = 0; i < threads->thread_count(); ++i) { MinidumpThread *thread = threads->GetThreadAtIndex(i); -- cgit v1.2.1