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

Unified Diff: session_manager_main.cc

Issue 553016: port to use centralized constants files, and add input validation (Closed)
Patch Set: address shell injection, otehr comments (per wad) Created 10 years, 11 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 | « mock_system_utils.h ('k') | session_manager_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: session_manager_main.cc
diff --git a/session_manager_main.cc b/session_manager_main.cc
index 29bac385243797548cd915f20d652f08c7acf751..ef1ab30747a7e50b306f2e67338731ca6e8840f3 100644
--- a/session_manager_main.cc
+++ b/session_manager_main.cc
@@ -16,6 +16,7 @@
#include "login_manager/child_job.h"
#include "login_manager/file_checker.h"
#include "login_manager/session_manager_service.h"
+#include "login_manager/system_utils.h"
// Watches a Chrome binary and restarts it when it crashes.
// Can also create and watch a named pipe and respond to IPCs defined
@@ -87,8 +88,7 @@ int main(int argc, char* argv[]) {
child_job->set_uid(uid);
::g_type_init();
- login_manager::SessionManagerService manager(child_job,
- false); // FOR NOW.
+ login_manager::SessionManagerService manager(child_job);
LOG_IF(FATAL, !manager.Initialize()) << "Failed";
LOG_IF(FATAL, !manager.Register(chromeos::dbus::GetSystemBusConnection()))
« no previous file with comments | « mock_system_utils.h ('k') | session_manager_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698