Fix ReadRemoteData in ModuleEventSinkImpl

chrome/browser/conflicts/module_event_sink_impl_win.cc

Index: chrome/browser/conflicts/module_event_sink_impl_win.cc
diff --git a/chrome/browser/conflicts/module_event_sink_impl_win.cc b/chrome/browser/conflicts/module_event_sink_impl_win.cc
index 808940332b0fbb6de9756415beeacd54b48726bc..d58475bab49427b6b1a8fc64e49d2c98e2e6dfd2 100644
--- a/chrome/browser/conflicts/module_event_sink_impl_win.cc
+++ b/chrome/browser/conflicts/module_event_sink_impl_win.cc
@@ -64,7 +64,7 @@ bool ReadRemoteData(base::ProcessHandle process, uint64_t address, T* data) {
   const void* typed_address =
       reinterpret_cast<const void*>(static_cast<uintptr_t>(address));
   SIZE_T bytes_read = 0;
-  if (!::ReadProcessMemory(process, typed_address, &data, sizeof(data),
+  if (!::ReadProcessMemory(process, typed_address, data, sizeof(data),

[Nico, 2017/02/15 15:35:09]

This still looks a bit funny. sizeof(data) will always be size of a pointer.
While &data was passed in, that was consistent, but now that you pass data do
you want sizeof(T) here?

                            &bytes_read)) {
     return false;
   }