diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2013-08-21 11:13:56 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-08-22 20:05:14 +0200 |
commit | 6f6f757d31ddbab13e5b4c2c1aad3e7d0dae722a (patch) | |
tree | 1822c777907ffb6ed0f30cdd2ebdf0d63422f8bb /src/util | |
parent | 38bf0c9a44ff9dccf38a2ed62a0ad2ad8b58c44e (diff) | |
download | sssd-6f6f757d31ddbab13e5b4c2c1aad3e7d0dae722a.tar.gz sssd-6f6f757d31ddbab13e5b4c2c1aad3e7d0dae722a.tar.xz sssd-6f6f757d31ddbab13e5b4c2c1aad3e7d0dae722a.zip |
Use brackets around macros.
warnings reported by cppcheck.
Diffstat (limited to 'src/util')
-rw-r--r-- | src/util/child_common.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/util/child_common.c b/src/util/child_common.c index ec2baf019..2bd2bbc3a 100644 --- a/src/util/child_common.c +++ b/src/util/child_common.c @@ -553,22 +553,22 @@ void child_sig_handler(struct tevent_context *ev, } else if (ret == 0) { DEBUG(1, ("waitpid did not found a child with changed status.\n")); } else { - if WIFEXITED(child_ctx->child_status) { + if (WIFEXITED(child_ctx->child_status)) { if (WEXITSTATUS(child_ctx->child_status) != 0) { DEBUG(1, ("child [%d] failed with status [%d].\n", ret, WEXITSTATUS(child_ctx->child_status))); } else { DEBUG(4, ("child [%d] finished successfully.\n", ret)); } - } else if WIFSIGNALED(child_ctx->child_status) { + } else if (WIFSIGNALED(child_ctx->child_status)) { DEBUG(1, ("child [%d] was terminated by signal [%d].\n", ret, WTERMSIG(child_ctx->child_status))); } else { - if WIFSTOPPED(child_ctx->child_status) { + if (WIFSTOPPED(child_ctx->child_status)) { DEBUG(7, ("child [%d] was stopped by signal [%d].\n", ret, WSTOPSIG(child_ctx->child_status))); } - if WIFCONTINUED(child_ctx->child_status) { + if (WIFCONTINUED(child_ctx->child_status)) { DEBUG(7, ("child [%d] was resumed by delivery of SIGCONT.\n", ret)); } |