summaryrefslogtreecommitdiff
path: root/source3/libsmb
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2002-10-26 00:59:21 +0000
committerAndrew Bartlett <abartlet@samba.org>2002-10-26 00:59:21 +0000
commitdbfc2f42a2c4711f539364d3954b9b9c10dbddf5 (patch)
tree7bfa38b98b198c7ac0b5bba80b1e7fd6e03da321 /source3/libsmb
parent185a232ae0e8e9d459cbaa74fee96f7357cf5e6c (diff)
downloadsamba-dbfc2f42a2c4711f539364d3954b9b9c10dbddf5.tar.gz
samba-dbfc2f42a2c4711f539364d3954b9b9c10dbddf5.tar.bz2
samba-dbfc2f42a2c4711f539364d3954b9b9c10dbddf5.zip
Make sure that we always return False if the password change never returns.
(A stuck 'unix passowrd sync' was gettting reported as 'success', causing all sorts of fun) Andrew Bartlett (This used to be commit b0d5745bb53b16e92160a6bdd2b984afb0e982ee)
Diffstat (limited to 'source3/libsmb')
-rw-r--r--source3/libsmb/clirap.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/source3/libsmb/clirap.c b/source3/libsmb/clirap.c
index 2064e14954..b4924fd773 100644
--- a/source3/libsmb/clirap.c
+++ b/source3/libsmb/clirap.c
@@ -343,13 +343,17 @@ BOOL cli_oem_change_password(struct cli_state *cli, const char *user, const char
return False;
}
- if (cli_receive_trans(cli,SMBtrans,
+ if (!cli_receive_trans(cli,SMBtrans,
&rparam, &rprcnt,
&rdata, &rdrcnt)) {
- if (rparam)
- cli->rap_error = SVAL(rparam,0);
+ DEBUG(0,("cli_oem_change_password: Failed to recieve reply to password change for user %s\n",
+ user ));
+ return False;
}
-
+
+ if (rparam)
+ cli->rap_error = SVAL(rparam,0);
+
SAFE_FREE(rparam);
SAFE_FREE(rdata);