From 56454f6fb2aed2bbfd6e1c1759f15d97db086d90 Mon Sep 17 00:00:00 2001 From: Diego Santa Cruz Date: Tue, 2 Feb 2016 13:08:05 +0100 Subject: bitbake: bitbake: prserv: do not clear umask when daemonizing Clearing the umask when daemonizing is not the correct thing to do, as it will create files writable by anyone by default. For instance the pid file was being created with mode 777. This could also potentially affect the sqlite database. Better let the calling process decide on the umask. [YOCTO #9036] (Bitbake rev: ff6d3f53a4504eae7ec4c190b9f7595b09aed017) Signed-off-by: Diego Santa Cruz Signed-off-by: Richard Purdie --- bitbake/lib/prserv/serv.py | 1 - 1 file changed, 1 deletion(-) (limited to 'bitbake/lib/prserv/serv.py') diff --git a/bitbake/lib/prserv/serv.py b/bitbake/lib/prserv/serv.py index 94d2abc15d..f588f4dc27 100644 --- a/bitbake/lib/prserv/serv.py +++ b/bitbake/lib/prserv/serv.py @@ -216,7 +216,6 @@ class PRServer(SimpleXMLRPCServer): def cleanup_handles(self): signal.signal(signal.SIGINT, self.sigint_handler) signal.signal(signal.SIGTERM, self.sigterm_handler) - os.umask(0) os.chdir("/") sys.stdout.flush() -- cgit v1.2.3-54-g00ecf