diff options
author | Elrond <elrond@samba.org> | 2000-03-30 19:14:08 +0000 |
---|---|---|
committer | Elrond <elrond@samba.org> | 2000-03-30 19:14:08 +0000 |
commit | 34cef0ace166aac85ba4eeee40d1ecad346a61d4 (patch) | |
tree | eba7b6b162d3b06b585fd136b787bf9c2d05beae | |
parent | ed84737946ea3f2db522015b15cc6d98177de5bc (diff) | |
download | samba-34cef0ace166aac85ba4eeee40d1ecad346a61d4.tar.gz |
Tims SMB_ASSERT for checking, that cli_auth_fns functions
are NULL just broke my anonymous lsaq.
I undid two of his things, until at least that worked
again.
(Look at rpc_client/cli_pipe_noauth.c, 3 of 5 functions are
NULL here.)
Luke, can you take a look at the other SMB_ASSERTs?
Elrond
-rw-r--r-- | source/rpc_client/cli_pipe.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/source/rpc_client/cli_pipe.c b/source/rpc_client/cli_pipe.c index c6ed46075ce..283cea7a66e 100644 --- a/source/rpc_client/cli_pipe.c +++ b/source/rpc_client/cli_pipe.c @@ -357,11 +357,10 @@ BOOL rpc_api_pipe_req(struct cli_connection *con, uint8 opnum, smb_io_rpc_hdr_resp("rpc_hdr_resp", &rhdr_resp, &rpdu, 0); } - SMB_ASSERT(auth->cli_decode_pdu != NULL); - - if (rhdr.auth_len != 0 && - !auth->cli_decode_pdu(con, &rpdu, rhdr.frag_len, - rhdr.auth_len)) + if (rhdr.auth_len != 0 + && (auth->cli_decode_pdu != NULL) + && !auth->cli_decode_pdu(con, &rpdu, rhdr.frag_len, + rhdr.auth_len)) { DEBUG(10, ("auth->cli_decode_pdu: failed\n")); return False; @@ -1006,10 +1005,8 @@ BOOL rpc_pipe_bind(struct cli_connection *con, nt->max_recv_frag = hdr_ba.bba.max_rsize; } - if (valid_ack) + if (valid_ack && auth->decode_bind_resp != NULL) { - SMB_ASSERT(auth->decode_bind_resp != NULL); - valid_ack = auth->decode_bind_resp(con, &rdata); if (valid_ack) |