summaryrefslogtreecommitdiffstats
path: root/src/util/child_common.c
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2014-09-15 16:05:30 +0200
committerJakub Hrozek <jhrozek@redhat.com>2015-02-11 20:38:42 +0100
commit5085d263f2f084778b1314fc5e808668c3758d82 (patch)
treeda7a12dd45222166481c225963bfedca23aaa115 /src/util/child_common.c
parent5594736ea2618bb3e487f47fd199e1d2cf4c58fd (diff)
downloadsssd-5085d263f2f084778b1314fc5e808668c3758d82.tar.gz
sssd-5085d263f2f084778b1314fc5e808668c3758d82.tar.xz
sssd-5085d263f2f084778b1314fc5e808668c3758d82.zip
Fix warning: equality comparison with extraneous parentheses
Example of warning: src/sss_client/libwbclient/wbc_pwd_sssd.c:246:23: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality] if (((wbc_status) == WBC_ERR_SUCCESS)) { ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ src/sss_client/libwbclient/wbc_pwd_sssd.c:246:23: note: remove extraneous parentheses around the comparison to silence this warning if (((wbc_status) == WBC_ERR_SUCCESS)) { ~ ^ ~ src/sss_client/libwbclient/wbc_pwd_sssd.c:246:23: note: use '=' to turn this equality comparison into an assignment if (((wbc_status) == WBC_ERR_SUCCESS)) { ^~ = The reason is definition of some macros which were used in if conditions. Reviewed-by: Michal Židek <mzidek@redhat.com>
Diffstat (limited to 'src/util/child_common.c')
-rw-r--r--src/util/child_common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/util/child_common.c b/src/util/child_common.c
index b1af02337..a6131cd20 100644
--- a/src/util/child_common.c
+++ b/src/util/child_common.c
@@ -568,7 +568,7 @@ static void child_sig_handler(struct tevent_context *ev,
"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) == true) {
DEBUG(SSSDBG_TRACE_LIBS,
"child [%d] was resumed by delivery of SIGCONT.\n",
ret);