From f84a9a6529467de33cef675d1c7336d207460113 Mon Sep 17 00:00:00 2001 From: Eric Lippmann Date: Tue, 30 Dec 2014 15:33:01 +0100 Subject: [PATCH] cli/setup: Do not use umask(0) chmod is not affected in any way by the umask. --- modules/setup/application/clicommands/ConfigCommand.php | 2 -- 1 file changed, 2 deletions(-) diff --git a/modules/setup/application/clicommands/ConfigCommand.php b/modules/setup/application/clicommands/ConfigCommand.php index a1cf3c19a..d3b2900bc 100644 --- a/modules/setup/application/clicommands/ConfigCommand.php +++ b/modules/setup/application/clicommands/ConfigCommand.php @@ -53,9 +53,7 @@ class ConfigCommand extends Command return false; } - $old = umask(0); // Prevent $mode from being mangled by the system's umask ($old) chmod($config, $mode); - umask($old); if (chgrp($config, $group) === false) { $this->fail(sprintf($this->translate('Unable to change the group of "%s" to "%s".'), $config, $group));