diff options
| author | Petr Cech <pcech@redhat.com> | 2015-10-05 10:12:36 -0400 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-10-14 13:27:13 +0200 |
| commit | f8e337540d280f944098cd4dd7d670e2f7166b54 (patch) | |
| tree | 1f1745c2a6859e968f2431a25c0f9d1f2f915764 /src/tests | |
| parent | d9c2a21119a6d04203060ad54fa8d20f17f5c0b7 (diff) | |
| download | sssd-f8e337540d280f944098cd4dd7d670e2f7166b54.tar.gz sssd-f8e337540d280f944098cd4dd7d670e2f7166b54.tar.xz sssd-f8e337540d280f944098cd4dd7d670e2f7166b54.zip | |
REFACTOR: umask(077) --> umask(SSS_DFL_X_UMASK)
There are many calls of umask function with 077 argument. This patch
add new constant SSS_DFL_X_UMASK which stands fot 077. So all
occurences of umask(077) are replaced by constant SSS_DFL_X_UMASK.
Resolves:
https://fedorahosted.org/sssd/ticket/2424
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Diffstat (limited to 'src/tests')
| -rw-r--r-- | src/tests/check_and_open-tests.c | 2 | ||||
| -rw-r--r-- | src/tests/debug-tests.c | 4 | ||||
| -rw-r--r-- | src/tests/util-tests.c | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/src/tests/check_and_open-tests.c b/src/tests/check_and_open-tests.c index e5981c858..25aee1fbf 100644 --- a/src/tests/check_and_open-tests.c +++ b/src/tests/check_and_open-tests.c @@ -48,7 +48,7 @@ void setup_check_and_open(void) filename = strdup(FILENAME_TEMPLATE); fail_unless(filename != NULL, "strdup failed"); - old_umask = umask(077); + old_umask = umask(SSS_DFL_X_UMASK); ret = mkstemp(filename); umask(old_umask); fail_unless(ret != -1, "mkstemp failed [%d][%s]", errno, strerror(errno)); diff --git a/src/tests/debug-tests.c b/src/tests/debug-tests.c index 067209b1d..8d9274014 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(077); + old_umask = umask(SSS_DFL_X_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(077); + old_umask = umask(SSS_DFL_X_UMASK); fd = mkstemp(filename); umask(old_umask); if (fd == -1) { diff --git a/src/tests/util-tests.c b/src/tests/util-tests.c index bfdf07802..88c6727d4 100644 --- a/src/tests/util-tests.c +++ b/src/tests/util-tests.c @@ -593,7 +593,7 @@ void setup_atomicio(void) fail_unless(filename != NULL, "strdup failed"); atio_fd = -1; - old_umask = umask(077); + old_umask = umask(SSS_DFL_X_UMASK); ret = mkstemp(filename); umask(old_umask); fail_unless(ret != -1, "mkstemp failed [%d][%s]", errno, strerror(errno)); |