From 24f651095aeb30078ecedb6b61eb17b08db8a338 Mon Sep 17 00:00:00 2001 From: Diego Santa Cruz Date: Tue, 2 Feb 2016 13:08:05 +0100 Subject: [PATCH] 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. Signed-off-by: Diego Santa Cruz --- bitbake/lib/prserv/serv.py | 1 - 1 file changed, 1 deletion(-) diff --git a/bitbake/lib/prserv/serv.py b/bitbake/lib/prserv/serv.py index 1de37cf..f6f50ee 100644 --- a/bitbake/lib/prserv/serv.py +++ b/bitbake/lib/prserv/serv.py @@ -212,7 +212,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() -- 1.9.4