]>
Commit | Line | Data |
---|---|---|
c0119cfb AF |
1 | From 0abef6992dc342d443137f8a2ac6c01f490cecee Mon Sep 17 00:00:00 2001 |
2 | From: Christian Ambach <ambi@samba.org> | |
3 | Date: Wed, 20 Feb 2013 16:59:05 +0100 | |
4 | Subject: [PATCH] s3:rpc_client fix a crash | |
5 | ||
6 | state->cli->dc does not have to be set (e.g. when running | |
7 | net rpc join against an older Samba PDC), so check it before dereferencing it | |
8 | ||
9 | This fixes Bug 9669 - net rpc join crashes against a Samba 3.0.33 PDC | |
10 | ||
11 | Bug: https://bugzilla.samba.org/show_bug.cgi?id=9669 | |
12 | ||
13 | Signed-off-by: Christian Ambach <ambi@samba.org> | |
14 | Reviewed-by: Andreas Schneider <asn@samba.org> | |
15 | ||
16 | Autobuild-User(master): Christian Ambach <ambi@samba.org> | |
17 | Autobuild-Date(master): Wed Feb 20 19:00:52 CET 2013 on sn-devel-104 | |
18 | (cherry picked from commit 3d29bb2d37b02909ecb500e864f3c13e06957a86) | |
19 | ||
20 | (cherry picked from commit ff658bb36c28c9db91fc80a68725e893ffe300aa) | |
21 | --- | |
22 | source3/rpc_client/cli_pipe.c | 2 +- | |
23 | 1 file changed, 1 insertion(+), 1 deletion(-) | |
24 | ||
25 | diff --git a/source3/rpc_client/cli_pipe.c b/source3/rpc_client/cli_pipe.c | |
26 | index 5ddabb7..a211d92 100644 | |
27 | --- a/source3/rpc_client/cli_pipe.c | |
28 | +++ b/source3/rpc_client/cli_pipe.c | |
29 | @@ -2136,7 +2136,7 @@ static void rpc_pipe_bind_step_two_done(struct tevent_req *subreq) | |
30 | status = dcerpc_netr_LogonGetCapabilities_r_recv(subreq, talloc_tos()); | |
31 | TALLOC_FREE(subreq); | |
32 | if (NT_STATUS_EQUAL(status, NT_STATUS_RPC_PROCNUM_OUT_OF_RANGE)) { | |
33 | - if (state->cli->dc->negotiate_flags & | |
34 | + if (state->cli->dc && state->cli->dc->negotiate_flags & | |
35 | NETLOGON_NEG_SUPPORTS_AES) { | |
36 | DEBUG(5, ("AES is not supported and the error was %s\n", | |
37 | nt_errstr(status))); | |
38 | -- | |
39 | 1.9.1 | |
40 |