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

Unified Diff: mojo/edk/system/handle_table.h

Issue 2012613002: Enforce/require MOJO_HANDLE_RIGHT_TRANSFER in sending handles via MojoWriteMessage(). (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 7 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: mojo/edk/system/handle_table.h
diff --git a/mojo/edk/system/handle_table.h b/mojo/edk/system/handle_table.h
index 3b8e967218f4c350ad03ff3346ba8143cf2333e0..e1d91a03ee8db15549c5e3dd64c00d3b71e22e76 100644
--- a/mojo/edk/system/handle_table.h
+++ b/mojo/edk/system/handle_table.h
@@ -75,8 +75,9 @@ class HandleTable {
bool AddHandleVector(HandleVector* handles, MojoHandle* handle_values);
// Tries to mark the given handle values as busy and start transport on them
- // (i.e., take their dispatcher locks); |transports| must be sized to contain
- // |num_handles| elements. On failure, returns them to their original
+ // (i.e., take their dispatcher locks). The handles to be transported must all
+ // have the |MOJO_HANDLE_RIGHT_TRANSFER| right. |transports| must be sized to
+ // contain |num_handles| elements. On failure, returns them to their original
// (non-busy, unlocked state).
MojoResult MarkBusyAndStartTransport(
MojoHandle disallowed_handle,

Powered by Google App Engine
This is Rietveld 408576698