diff options
| author | Petr Cech <pcech@redhat.com> | 2015-10-06 09:51:21 -0400 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-10-14 13:27:27 +0200 |
| commit | c4a1191d673e4368f1831cbeb4d75b15e51ff6db (patch) | |
| tree | 5b7c32cb47b4e7e5085461c6951d0ee9143564ff | |
| parent | 4230e6faa907b693ccf66b3a29f9be57b87aab14 (diff) | |
| download | sssd-c4a1191d673e4368f1831cbeb4d75b15e51ff6db.tar.gz sssd-c4a1191d673e4368f1831cbeb4d75b15e51ff6db.tar.xz sssd-c4a1191d673e4368f1831cbeb4d75b15e51ff6db.zip | |
TESTS: More restrictive permissions in debug_tests
Debug tests try to write into and read from crreated files. There is no
reason to have executable permission, so this patch replaces
SSS_DFl_X_UMASK with SSS_DFL_UMASK permissions.
Resolves:
https://fedorahosted.org/sssd/ticket/2424
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
| -rw-r--r-- | src/tests/debug-tests.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/tests/debug-tests.c b/src/tests/debug-tests.c index 8d9274014..d904d7eb8 100644 --- a/src/tests/debug-tests.c +++ b/src/tests/debug-tests.c @@ -133,7 +133,7 @@ int test_helper_debug_check_message(int level) strncpy(filename, "sssd_debug_tests.XXXXXX", 24); - old_umask = umask(SSS_DFL_X_UMASK); + old_umask = umask(SSS_DFL_UMASK); fd = mkstemp(filename); umask(old_umask); if (fd == -1) { @@ -273,7 +273,7 @@ int test_helper_debug_is_empty_message(int level) strncpy(filename, "sssd_debug_tests.XXXXXX", 24); - old_umask = umask(SSS_DFL_X_UMASK); + old_umask = umask(SSS_DFL_UMASK); fd = mkstemp(filename); umask(old_umask); if (fd == -1) { |