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

Unified Diff: sandbox/mac/launchd_interception_server.h

Issue 332523003: Attempt to fix a NULL deref in sandbox::LaunchdInterceptionServer::HandleLookUp(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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
« no previous file with comments | « no previous file | sandbox/mac/launchd_interception_server.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sandbox/mac/launchd_interception_server.h
diff --git a/sandbox/mac/launchd_interception_server.h b/sandbox/mac/launchd_interception_server.h
index f45c74fc877ead44e8389f4cefbb194bdd5ec503..2e80fecaf5e40f8f65810bf0eb17d076c42722ff 100644
--- a/sandbox/mac/launchd_interception_server.h
+++ b/sandbox/mac/launchd_interception_server.h
@@ -16,6 +16,7 @@
namespace sandbox {
class BootstrapSandbox;
+struct BootstrapSandboxPolicy;
// This class is used to run a Mach IPC message server. This server can
// hold the receive right for a bootstrap_port of a process, and it filters
@@ -41,14 +42,13 @@ class LaunchdInterceptionServer : public MessageDemuxer {
// policy for the service name then formulates and sends the reply message.
void HandleLookUp(mach_msg_header_t* request,
mach_msg_header_t* reply,
- pid_t sender_pid);
+ const BootstrapSandboxPolicy* policy);
// Given a swap_integer request message, this verifies that it is safe, and
// if so, forwards it on to launchd for servicing. If the request is unsafe,
// it replies with an error.
void HandleSwapInteger(mach_msg_header_t* request,
- mach_msg_header_t* reply,
- pid_t sender_pid);
+ mach_msg_header_t* reply);
// Forwards the original |request| on to real bootstrap server for handling.
void ForwardMessage(mach_msg_header_t* request);
« no previous file with comments | « no previous file | sandbox/mac/launchd_interception_server.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698