s3:rpc_client fix a crash
authorChristian Ambach <ambi@samba.org>
Wed, 20 Feb 2013 15:59:05 +0000 (16:59 +0100)
committerKarolin Seeger <kseeger@samba.org>
Tue, 26 Feb 2013 07:50:26 +0000 (08:50 +0100)
state->cli->dc does not have to be set (e.g. when running
net rpc join against an older Samba PDC), so check it before dereferencing it

This fixes Bug 9669 - net rpc join crashes against a Samba 3.0.33 PDC

Bug: https://bugzilla.samba.org/show_bug.cgi?id=9669

Signed-off-by: Christian Ambach <ambi@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Christian Ambach <ambi@samba.org>
Autobuild-Date(master): Wed Feb 20 19:00:52 CET 2013 on sn-devel-104
(cherry picked from commit 3d29bb2d37b02909ecb500e864f3c13e06957a86)

source3/rpc_client/cli_pipe.c

index 61e6cce69618810a18715f8f70a563af35eabca0..24d69ff41ad04cab4a59b737c1f9e07e9d866d92 100644 (file)
@@ -1805,7 +1805,7 @@ static void rpc_pipe_bind_step_two_done(struct tevent_req *subreq)
        status = dcerpc_netr_LogonGetCapabilities_r_recv(subreq, talloc_tos());
        TALLOC_FREE(subreq);
        if (NT_STATUS_EQUAL(status, NT_STATUS_RPC_PROCNUM_OUT_OF_RANGE)) {
-               if (state->cli->dc->negotiate_flags &
+               if (state->cli->dc && state->cli->dc->negotiate_flags &
                    NETLOGON_NEG_SUPPORTS_AES) {
                        DEBUG(5, ("AES is not supported and the error was %s\n",
                                  nt_errstr(status)));