diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2016-02-22 10:56:52 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2016-02-25 12:52:06 +0100 |
| commit | 401d8b0600dd2d36f6d62ee1d72f56a245cc3158 (patch) | |
| tree | 5d7d748e858cd6570fd5fdb56cbddf34f94bbb68 | |
| parent | faa16fc9f0c9a02b26497e7cf148a92586144c08 (diff) | |
| download | sssd-401d8b0600dd2d36f6d62ee1d72f56a245cc3158.tar.gz sssd-401d8b0600dd2d36f6d62ee1d72f56a245cc3158.tar.xz sssd-401d8b0600dd2d36f6d62ee1d72f56a245cc3158.zip | |
IPA: Use the common if-else coding style
Reviewed-by: Petr Cech <pcech@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
| -rw-r--r-- | src/providers/ipa/ipa_subdomains_ext_groups.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/providers/ipa/ipa_subdomains_ext_groups.c b/src/providers/ipa/ipa_subdomains_ext_groups.c index 5dc6d0d64..149e5b5df 100644 --- a/src/providers/ipa/ipa_subdomains_ext_groups.c +++ b/src/providers/ipa/ipa_subdomains_ext_groups.c @@ -80,8 +80,7 @@ static errno_t process_ext_groups(TALLOC_CTX *mem_ctx, size_t reply_count, if (ret == ENOENT) { /* no external members, try next external group. */ continue; - } - if (ret != EOK) { + } else if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_get_string_array failed.\n"); goto done; @@ -92,8 +91,7 @@ static errno_t process_ext_groups(TALLOC_CTX *mem_ctx, size_t reply_count, if (ret == ENOENT) { /* no IPA groups, try next external group. */ continue; - } - if (ret != EOK) { + } else if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_get_string_array failed.\n"); goto done; |