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

Unified Diff: ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc

Issue 14750007: NaCl: enable meta-based validation for shared libraries. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More edits Created 7 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: ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc
diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc b/ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc
index 836b0861e1a11543e1464418c45f64c97640800f..a732472c8b5511cdc14ab97aec6c606468b86902 100644
--- a/ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc
+++ b/ppapi/native_client/src/trusted/plugin/pnacl_coordinator.cc
@@ -263,8 +263,8 @@ int32_t PnaclCoordinator::GetLoadedFileDesc(int32_t pp_error,
NACL_PRId32", url=%s, component=%s)\n", pp_error,
url.c_str(), component.c_str()));
ErrorInfo error_info;
- int32_t file_desc_ok_to_close = plugin_->GetPOSIXFileDesc(url);
- if (pp_error != PP_OK || file_desc_ok_to_close == NACL_NO_FILE_DESC) {
+ struct NaClFileInfo info = plugin_->GetFileInfo(url);
+ if (pp_error != PP_OK || info.desc == NACL_NO_FILE_DESC) {
if (pp_error == PP_ERROR_ABORTED) {
plugin_->ReportLoadAbort();
} else {
@@ -274,7 +274,7 @@ int32_t PnaclCoordinator::GetLoadedFileDesc(int32_t pp_error,
}
return NACL_NO_FILE_DESC;
}
- return file_desc_ok_to_close;
+ return info.desc;
}
PnaclCoordinator::PnaclCoordinator(

Powered by Google App Engine
This is Rietveld 408576698