diff options
| author | Werner Koch <[email protected]> | 2020-12-30 14:00:28 +0000 |
|---|---|---|
| committer | Werner Koch <[email protected]> | 2020-12-30 14:06:33 +0000 |
| commit | fdc54850263b2b888398f95be7816134b45a60d3 (patch) | |
| tree | 6ddabaacc0595108a914fc022110dcb8fd25b0d6 | |
| parent | gpg: Initialize a variable even in a never used code path. (diff) | |
| download | gnupg-fdc54850263b2b888398f95be7816134b45a60d3.tar.gz gnupg-fdc54850263b2b888398f95be7816134b45a60d3.zip | |
wkd: Minor permission fix for created files.
* tools/wks-util.c (wks_cmd_install_key): Don't set u+x on the file.
(ensure_policy_file): No need to make the policy file group writable.
--
The policy file is rarely changed thus no need to g+w. Setting +x on
a plain file does not make sense at all.
GnuPG-bug-id: 5214
(cherry picked from commit c008e8d20e12c8845403ad7dad499f6a196ecc6a)
| -rw-r--r-- | tools/wks-util.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/wks-util.c b/tools/wks-util.c index 085301fea..8090a7eaa 100644 --- a/tools/wks-util.c +++ b/tools/wks-util.c @@ -931,7 +931,7 @@ ensure_policy_file (const char *addrspec) log_info ("policy file '%s' created\n", fname); /* Make sure the policy file world readable. */ - if (gnupg_chmod (fname, "-rw-rw-r--")) + if (gnupg_chmod (fname, "-rw-r--r--")) { err = gpg_error_from_syserror (); log_error ("can't set permissions of '%s': %s\n", @@ -1132,7 +1132,7 @@ wks_cmd_install_key (const char *fname, const char *userid) } /* Make sure it is world readable. */ - if (gnupg_chmod (huname, "-rwxr--r--")) + if (gnupg_chmod (huname, "-rw-r--r--")) log_error ("can't set permissions of '%s': %s\n", huname, gpg_strerror (gpg_err_code_from_syserror())); |