From 81c6415aa14d86630531d741a1c4c4f552cb84af Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Thu, 2 Jan 2003 13:01:44 +0000 Subject: Becouse these functions return a size_t, they can't return negitive numbers. So a < 0 check is pointless, instead check against -1, which will be cast to unsigned. Andrew Bartlett (This used to be commit c7e5797a95804d5779cdfc93795adcdf2fe82a59) --- source3/nsswitch/winbindd_pam.c | 6 +++--- source3/smbd/mangle_hash.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'source3') diff --git a/source3/nsswitch/winbindd_pam.c b/source3/nsswitch/winbindd_pam.c index c5b68ea058..398b001f8b 100644 --- a/source3/nsswitch/winbindd_pam.c +++ b/source3/nsswitch/winbindd_pam.c @@ -185,13 +185,13 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state) goto done; } - if (pull_utf8_talloc(mem_ctx, &user, state->request.data.auth_crap.user) < 0) { + if (pull_utf8_talloc(mem_ctx, &user, state->request.data.auth_crap.user) == -1) { DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n")); } if (*state->request.data.auth_crap.domain) { char *dom = NULL; - if (pull_utf8_talloc(mem_ctx, &dom, state->request.data.auth_crap.domain) < 0) { + if (pull_utf8_talloc(mem_ctx, &dom, state->request.data.auth_crap.domain) == -1) { DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n")); } domain = dom; @@ -215,7 +215,7 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state) if (*state->request.data.auth_crap.workstation) { char *wrk = NULL; - if (pull_utf8_talloc(mem_ctx, &wrk, state->request.data.auth_crap.workstation) < 0) { + if (pull_utf8_talloc(mem_ctx, &wrk, state->request.data.auth_crap.workstation) == -1) { DEBUG(0, ("winbindd_pam_auth_crap: pull_utf8_talloc failed!\n")); } workstation = wrk; diff --git a/source3/smbd/mangle_hash.c b/source3/smbd/mangle_hash.c index 929cb0e07f..ac9c13dcac 100644 --- a/source3/smbd/mangle_hash.c +++ b/source3/smbd/mangle_hash.c @@ -730,7 +730,7 @@ static void name_map(char *OutName, BOOL need83, BOOL cache83) DEBUG(5,("name_map( %s, need83 = %s, cache83 = %s)\n", OutName, need83 ? "True" : "False", cache83 ? "True" : "False")); - if (push_ucs2_allocate(&OutName_ucs2, OutName) < 0) { + if (push_ucs2_allocate(&OutName_ucs2, OutName) == -1 ) { DEBUG(0, ("push_ucs2_allocate failed!\n")); return; } -- cgit