Don't call uninitialized sandbox helper function from blacklist intercepts.

When mapping a section into a different process, the IsSameProcess
function would crash invoking an uninitialized nt export.

BUG=330178
TEST=NONE

Review URL: https://codereview.chromium.org/93513011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242272 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 1 deletions

diff --git a/chrome_elf/blacklist/blacklist_interceptions.cc b/chrome_elf/blacklist/blacklist_interceptions.cc
index f1e5b30..d0f64ea 100644
--- a/chrome_elf/blacklist/blacklist_interceptions.cc
+++ b/chrome_elf/blacklist/blacklist_interceptions.cc
@@ -161,6 +161,11 @@ base::string16 GetImageInfoFromLoadedModule(HMODULE module, uint32* flags) {
   return base::string16(out_name.begin(), out_name.end());
 }
 
+bool IsSameAsCurrentProcess(HANDLE process) {
+  return  (NtCurrentProcess == process) ||
+          (::GetProcessId(process) == ::GetCurrentProcessId());
+}
+
 }  // namespace
 
 namespace blacklist {
@@ -195,7 +200,7 @@ SANDBOX_INTERCEPT NTSTATUS WINAPI BlNtMapViewOfSection(
                                        commit_size, offset, view_size, inherit,
                                        allocation_type, protect);
 
-  if (!NT_SUCCESS(ret) || !sandbox::IsSameProcess(process) ||
+  if (!NT_SUCCESS(ret) || !IsSameAsCurrentProcess(process) ||
       !IsModuleValidImageSection(section, base, offset, view_size)) {
     return ret;
   }