diff options
| author |   Diego Santa Cruz <Diego.SantaCruz@spinetix.com> | 2016-02-02 13:08:05 +0100 |
|---|---|---|
| committer |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-02-04 23:46:01 +0000 |
| commit | ff6d3f53a4504eae7ec4c190b9f7595b09aed017 (patch) | |
| tree | 82549a51a0334dfd80f7c60a51d572c1aff37dd1 | |
| parent | 655ec800d54da581229f12efb6f0baf54975fed4 (diff) | |
| download | bitbake-ff6d3f53a4504eae7ec4c190b9f7595b09aed017.tar.gz | |
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]
Signed-off-by: Diego Santa Cruz <Diego.SantaCruz@spinetix.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | lib/prserv/serv.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/prserv/serv.py b/lib/prserv/serv.py index 94d2abc15..f588f4dc2 100644 --- a/lib/prserv/serv.py +++ b/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() |