diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2003-01-02 13:01:44 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2003-01-02 13:01:44 +0000 |
| commit | c7e5797a95804d5779cdfc93795adcdf2fe82a59 (patch) | |
| tree | 7db978dccf6aeda13ec4ebb0d71d4db3870d5787 /source/nsswitch | |
| parent | 23abf75adef2f400414dfafa1248f5f886ae27c0 (diff) | |
| download | samba-c7e5797a95804d5779cdfc93795adcdf2fe82a59.tar.gz samba-c7e5797a95804d5779cdfc93795adcdf2fe82a59.tar.xz samba-c7e5797a95804d5779cdfc93795adcdf2fe82a59.zip | |
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
Diffstat (limited to 'source/nsswitch')
| -rw-r--r-- | source/nsswitch/winbindd_pam.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/source/nsswitch/winbindd_pam.c b/source/nsswitch/winbindd_pam.c index c5b68ea0583..398b001f8bf 100644 --- a/source/nsswitch/winbindd_pam.c +++ b/source/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; |