Utility functions to interact with the NT loader's data structures and associ...

chrome_frame/crash_reporting/nt_loader.cc

Index: chrome_frame/crash_reporting/nt_loader.cc
===================================================================
--- chrome_frame/crash_reporting/nt_loader.cc	(revision 0)
+++ chrome_frame/crash_reporting/nt_loader.cc	(revision 0)
@@ -0,0 +1,29 @@
+// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+#include "chrome_frame/crash_reporting/nt_loader.h"
+
+#include <winnt.h>
+#include <winternl.h>
+#include "base/logging.h"
+
+namespace nt_loader {
+
+LDR_DATA_TABLE_ENTRY* GetLoaderEntry(HMODULE module) {
+  // Make sure we own the loader's lock on entry here.
+  DCHECK(OwnsCriticalSection(GetLoaderLock()));

[amit, 2010/03/11 16:08:51]

Shouldn't this an 'if' instead? We probably do not want to go through the list
if loader lock is not held by us?

[Sigurður Ásgeirsson, 2010/03/11 16:16:35]

The DCHECK is asserting on the documented precondition of the function, that the
loader lock must be held on entry. We should never go in here without the lock
held, but if we ever do, I want the *developer* to be tripped immediately.

+  PEB* peb = GetCurrentPeb();
+
+  LIST_ENTRY* head = &peb->Ldr->InLoadOrderModuleList;
+  for (LIST_ENTRY* entry = head->Flink; entry != head; entry = entry->Flink) {
+    LDR_DATA_TABLE_ENTRY* ldr_entry =
+        CONTAINING_RECORD(entry, LDR_DATA_TABLE_ENTRY, InLoadOrderLinks);
+
+    if (reinterpret_cast<HMODULE>(ldr_entry->DllBase) == module)
+      return ldr_entry;
+  }
+
+  return NULL;
+}
+
+}  // namespace nt_loader
Property changes on: chrome_frame\crash_reporting\nt_loader.cc
___________________________________________________________________
Added: svn:eol-style
   + LF