Bug 1434822 part 1: Disable COM ping functionality for our parent process instead of for external processes. r?aklotz draft
authorJames Teh <jteh@mozilla.com>
Thu, 01 Feb 2018 15:42:24 +1000
changeset 758422 0a59b401074ffd64b580680abcd3101bef393d56
parent 756852 ad133cd410a719c0b67e61b8d3b1c77a32fd80a9
child 758423 f108b1a9cf95ac77d07213357a8b075cf237c76e
child 761503 bb35d1fefb1cb5d101952f335deed28e5d54fa6c
push id100046
push userbmo:jteh@mozilla.com
push dateThu, 22 Feb 2018 12:02:01 +0000
reviewersaklotz
bugs1434822
milestone60.0a1
Bug 1434822 part 1: Disable COM ping functionality for our parent process instead of for external processes. r?aklotz MozReview-Commit-ID: 2ayfzIxh8Lo
ipc/mscom/FastMarshaler.cpp
--- a/ipc/mscom/FastMarshaler.cpp
+++ b/ipc/mscom/FastMarshaler.cpp
@@ -98,17 +98,17 @@ FastMarshaler::InternalRelease()
 DWORD
 FastMarshaler::GetMarshalFlags(DWORD aDestContext, DWORD aMshlFlags)
 {
   // Only worry about local contexts.
   if (aDestContext != MSHCTX_LOCAL) {
     return aMshlFlags;
   }
 
-  if (!IsCallerExternalProcess()) {
+  if (IsCallerExternalProcess()) {
     return aMshlFlags;
   }
 
   // The caller is our parent main thread. Disable ping functionality.
   return aMshlFlags | MSHLFLAGS_NOPING;
 }
 
 HRESULT