huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #22833 from dotnet-maestro-bot/merge/release/3.1-to-master 

[automated] Merge branch 'release/3.1' => 'master'
This commit is contained in:
msftbot[bot] 2020-06-11 19:32:33 +00:00 committed by GitHub
parent 73f04eecac 812f2f8145
commit 62e6d93f5c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 70 additions and 61 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/Servers/IIS/AspNetCoreModuleV2/InProcessRequestHandler/inprocesshandler.cpp
View File

@ -112,7 +112,7 @@ IN_PROCESS_HANDLER::NotifyDisconnect()
if (pManagedHttpContext != nullptr)
{
m_pDisconnectHandler(m_pManagedHttpContext);
m_pDisconnectHandler(pManagedHttpContext);
}
}
@ -139,14 +139,17 @@ IN_PROCESS_HANDLER::SetManagedHttpContext(
PVOID pManagedHttpContext
)
{
bool disconnectFired = false;
{
SRWExclusiveLock lock(m_srwDisconnectLock);
m_pManagedHttpContext = pManagedHttpContext;
disconnectFired = m_disconnectFired;
}
if (m_disconnectFired && m_pManagedHttpContext != nullptr)
if (disconnectFired && pManagedHttpContext != nullptr)
{
m_pDisconnectHandler(m_pManagedHttpContext);
m_pDisconnectHandler(pManagedHttpContext);
}
}

6
src/Servers/IIS/IIS/src/Core/IISHttpContext.cs
View File

@ -120,6 +120,11 @@ namespace Microsoft.AspNetCore.Server.IIS.Core
public long? MaxRequestBodySize { get; set; }
protected void InitializeContext()
{
// create a memory barrier between initialize and disconnect to prevent a possible
// NullRef with disconnect being called before these fields have been written
// disconnect aquires this lock as well
lock (_abortLock)
{
_thisHandle = GCHandle.Alloc(this);
@ -184,6 +189,7 @@ namespace Microsoft.AspNetCore.Server.IIS.Core
resumeWriterThreshold: ResumeWriterTheshold,
minimumSegmentSize: MinAllocBufferSize));
_bodyOutput = new OutputProducer(pipe);
}
NativeMethods.HttpSetManagedContext(_pInProcessHandler, (IntPtr)_thisHandle);
}

4
src/Servers/IIS/IIS/src/Core/IISHttpServer.cs
View File

@ -170,7 +170,7 @@ namespace Microsoft.AspNetCore.Server.IIS.Core
try
{
context = (IISHttpContext)GCHandle.FromIntPtr(pvManagedHttpContext).Target;
context.AbortIO(clientDisconnect: true);
context?.AbortIO(clientDisconnect: true);
}
catch (Exception ex)
{
@ -184,7 +184,7 @@ namespace Microsoft.AspNetCore.Server.IIS.Core
try
{
context = (IISHttpContext)GCHandle.FromIntPtr(pvManagedHttpContext).Target;
context.OnAsyncCompletion(hr, bytes);
context?.OnAsyncCompletion(hr, bytes);
return NativeMethods.REQUEST_NOTIFICATION_STATUS.RQ_NOTIFICATION_PENDING;
}
catch (Exception ex)