[ovs-git] Open vSwitch: daemon: Start monitor process, not daemon process, in new session. (master)

[dev at openvswitch.org]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  066f329e29d5ff0b97ef77a5711d87ed53eef73d (commit)
      from  ed36537ebf48108accf21a8aa073279eceeafa98 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 066f329e29d5ff0b97ef77a5711d87ed53eef73d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=066f329e29d5ff0b97ef77a5711d87ed53eef73d
Author: Ben Pfaff <blp at nicira.com>
		
daemon: Start monitor process, not daemon process, in new session.
		
To keep control+C and other signals in the initiating session from killing
the monitor process, we need to put the monitor process into its own
session.  However, until this point, we've only done that for the daemon
processes that the monitor started, which means that control+C would kill
the monitor but not the daemons that it launched.

I don't know of a benefit to putting the monitor and daemon processes in
different sessions, as opposed to one new session for both of them, so
this change does the latter.

daemonize_post_detach() is called from one additional context where we'd
want to be in a new session, the worker_start() function, but that function
is documented as to be called after daemonize_start(), in which case we
will (after this commit) already have called setsid(), so no additional
change is required there.

Bug #14280.
Reported-by: Gordon Good <ggood at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 lib/daemon.c         |    3 ++-
 python/ovs/daemon.py |    5 +++--
 2 files changed, 5 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
Open vSwitch