google-breakpad: Fix test addresses to use uintptr_t instead of u_int64_t.
When a variable is used to set (and lookup) MappingInfo's "start_addr"
field, it needs to match types -- which is "uintptr_t". Updating the 'make'
that's used for building apparently caused the 32-bit "char *" to
sign-extend when cast up to a u_int64_t -- maybe because pointers were
unsigned before and then changed to be signed?
BUG=chromium-os:25355
TEST=Ran Breakpad unittests
Change-Id: I79ddd447e41e44a6d33ceec84a57671636132cdc
Reviewed-on: https://gerrit.chromium.org/gerrit/14859
Tested-by: Michael Krebs <mkrebs@chromium.org>
Commit-Ready: Michael Krebs <mkrebs@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
diff --git a/src/client/linux/minidump_writer/minidump_writer_unittest.cc b/src/client/linux/minidump_writer/minidump_writer_unittest.cc
index ee021ae..ef1e7a5 100644
--- a/src/client/linux/minidump_writer/minidump_writer_unittest.cc
+++ b/src/client/linux/minidump_writer/minidump_writer_unittest.cc
@@ -121,7 +121,7 @@
MAP_PRIVATE | MAP_ANON,
-1,
0));
- const u_int64_t kMemoryAddress = reinterpret_cast<u_int64_t>(memory);
+ const uintptr_t kMemoryAddress = reinterpret_cast<uintptr_t>(memory);
ASSERT_TRUE(memory);
const pid_t child = fork();
@@ -224,7 +224,7 @@
MAP_PRIVATE,
fd,
0));
- const u_int64_t kMemoryAddress = reinterpret_cast<u_int64_t>(memory);
+ const uintptr_t kMemoryAddress = reinterpret_cast<uintptr_t>(memory);
ASSERT_TRUE(memory);
close(fd);