Don't delete memory passed to openlog/syslog.

Fixes the problem where syslog wrote a blank tag instead of
"chromoting_me2me_host" (problem seen after applying
http://codereview.chromium.org/9234070/)


BUG=None
TEST=Manual


Review URL: http://codereview.chromium.org/9143003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119491 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 11 insertions, 2 deletions

diff --git a/remoting/host/system_event_logger_linux.cc b/remoting/host/system_event_logger_linux.cc
index 4518d5d..7100006 100644
--- a/remoting/host/system_event_logger_linux.cc
+++ b/remoting/host/system_event_logger_linux.cc
@@ -15,14 +15,23 @@ namespace {
 
 class SystemEventLoggerLinux : public SystemEventLogger {
  public:
-  SystemEventLoggerLinux(const std::string& application_name) {
-    openlog(application_name.c_str(), 0, LOG_USER);
+  SystemEventLoggerLinux(const std::string& application_name)
+      : application_name_(application_name) {
+    openlog(application_name_.c_str(), 0, LOG_USER);
+  }
+
+  ~SystemEventLoggerLinux() {
+    closelog();
   }
 
   virtual void Log(const std::string& message) OVERRIDE {
     syslog(LOG_USER | LOG_NOTICE, "%s", message.c_str());
   }
 
+ private:
+  // Store this string here, to avoid deleting memory passed to openlog().
+  std::string application_name_;
+
   DISALLOW_COPY_AND_ASSIGN(SystemEventLoggerLinux);
 };