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

Unified Diff: mojo/monacl/README

Issue 385983008: Mojo + NaCl prototype. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ClipDep Created 6 years, 3 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/monacl/README
diff --git a/mojo/monacl/README b/mojo/monacl/README
new file mode 100644
index 0000000000000000000000000000000000000000..65a2447e9f08c8f5dc08ee18ff042438fdf95cec
--- /dev/null
+++ b/mojo/monacl/README
@@ -0,0 +1,23 @@
+This is a prototype for plumbing Mojo into the NaCl sandbox. It is
+currently insecure (see below), does not provide a stable ABI (IRT
+support must be added), and does not support Mojo functions that
+return pointers (for example, MojoMapBuffer).
+
Mark Seaborn 2014/09/10 23:19:20 On naming: "monacl" is cutesy, but it's not necess
Nick Bray (chromium) 2014/09/11 00:56:39 Bulk moved. Some instances of "monacl" not change
+generator/interface.py contains a programmatic description of the
+stable Mojo interface. This will need to be updated as the interface
+changes. Run generator/generate_monacl_bindings.py to generate the
+bindings that plumb this interface into the NaCl sandbox.
+
+To test: Build "monacl_shell" and "monacl_test". Run monacl_shell
+with the IRT as the first argument and the monacl_test as the second
+argument. For example, to run a Debug 32-bit Intel build:
+
+ cd out/Debug
+ ./monacl_shell irt_core_newlib_x32.nexe monacl_test_newlib_x32.nexe
+
+Security TODO list:
+ Separate trusted and untrusted Mojo handles.
+ Validate and copy option structures.
+ Protect untrusted buffers passed into Mojo:
+ NaClVmIoWillStart/HasEnded.
+ Data races.
Mark Seaborn 2014/09/10 23:19:20 What do you mean by "data races"? Does this mean
Nick Bray (chromium) 2014/09/11 00:56:39 Done.

Powered by Google App Engine
This is Rietveld 408576698