Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6358)

Unified Diff: chrome/service/service_process.cc

Issue 2444793002: Allow custom security descriptors when creating named pipes on Windows. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/service/service_process.cc
diff --git a/chrome/service/service_process.cc b/chrome/service/service_process.cc
index 56d8db8e67e80b92ad5e7b6dc85251c53bc5f03a..68d4f82ae9a48bd4db9c75ee0969825daf6de9cf 100644
--- a/chrome/service/service_process.cc
+++ b/chrome/service/service_process.cc
@@ -131,7 +131,7 @@ mojo::edk::ScopedPlatformHandle CreateServerHandle(
return mojo::edk::ScopedPlatformHandle(platform_handle);
#else
return mojo::edk::CreateServerHandle(
- mojo::edk::NamedPlatformHandle(channel_handle.name), false);
+ mojo::edk::NamedPlatformHandle(channel_handle.name));
#endif
}
#endif
@@ -334,7 +334,9 @@ mojo::ScopedMessagePipeHandle ServiceProcess::CreateChannelMessagePipe() {
#if defined(OS_POSIX)
channel_handle = mojo::edk::DuplicatePlatformHandle(server_handle_.get());
#elif defined(OS_WIN)
- channel_handle = mojo::edk::CreateServerHandle(server_handle_, false);
+ mojo::edk::CreateServerHandleOptions options;
+ options.enforce_uniqueness = false;
+ channel_handle = mojo::edk::CreateServerHandle(server_handle_, options);
#endif
CHECK(channel_handle.is_valid());

Powered by Google App Engine
This is Rietveld 408576698