fix AV in failure path before request assignment to a worker process

Azure · Aug 23, 2011 · 41ab29c · 41ab29c
1 parent d63f678
commit 41ab29c
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/iisnode/cprotocolbridge.cpp b/src/iisnode/cprotocolbridge.cpp
@@ -20,7 +20,13 @@ HRESULT CProtocolBridge::SendEmptyResponse(CNodeHttpStoredContext* context, USHO
 
 	context->SetHresult(hresult);
 	context->SetRequestNotificationStatus(RQ_NOTIFICATION_FINISH_REQUEST);
-	context->GetNodeProcess()->OnRequestCompleted(context);
+	if (NULL != context->GetNodeProcess())
+	{
+		// there is no CNodeProcess assigned to the request yet - something failed before it was moved from the pending queue
+		// to an active request queue of a specific process
+
+		context->GetNodeProcess()->OnRequestCompleted(context);
+	}
 
 	CProtocolBridge::SendEmptyResponse(context->GetHttpContext(), status, reason, hresult);