From a6403ba4d9187fc19af8e3dc64ec3e9e937d53eb Mon Sep 17 00:00:00 2001
From: Mike McLaughlin <mikem@microsoft.com>
Date: Thu, 13 Dec 2018 15:37:21 -0800
Subject: Fixes issue #21485 and #21484 (#21526)

Issue #21485: fix EnumProcessModules hPseudoCurrentProcess bug.

Added handle reference.

Issue #21484: createdump segfaults with ASP.NET app

The problem is the ClrDataModule Request faulted on a dynamic module
getting the file layout flag.

Fixed the Request code not get the file layout and in the crash dump
code skip any dynamic modules.
---
 src/pal/src/thread/process.cpp | 1 +
 1 file changed, 1 insertion(+)

(limited to 'src/pal')

diff --git a/src/pal/src/thread/process.cpp b/src/pal/src/thread/process.cpp
index a964ee8829..dcaa307e36 100644
--- a/src/pal/src/thread/process.cpp
+++ b/src/pal/src/thread/process.cpp
@@ -2859,6 +2859,7 @@ GetProcessModulesFromHandle(
     if (hPseudoCurrentProcess == hProcess)
     {
         pobjProcess = g_pobjProcess;
+        pobjProcess->AddReference();
     }
     else
     {
-- 
cgit v1.2.3