diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2013-06-28 09:15:16 +1000 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2013-07-24 16:35:32 +0200 |
| commit | e9faf50ee123a8d1d647ebffa39107ca0dce756c (patch) | |
| tree | feed7761b718deb25f6ef60d2fc86312a6dffecf /source4 | |
| parent | ae0ba6bd833f71c4337ae3b6621bf797cb3c48c2 (diff) | |
| download | samba-e9faf50ee123a8d1d647ebffa39107ca0dce756c.tar.gz samba-e9faf50ee123a8d1d647ebffa39107ca0dce756c.tar.bz2 samba-e9faf50ee123a8d1d647ebffa39107ca0dce756c.zip | |
dsdb: Add assert in drepl_take_FSMO_role
Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source4')
| -rw-r--r-- | source4/dsdb/repl/drepl_fsmo.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/source4/dsdb/repl/drepl_fsmo.c b/source4/dsdb/repl/drepl_fsmo.c index 37fb684f47..7a107da6a3 100644 --- a/source4/dsdb/repl/drepl_fsmo.c +++ b/source4/dsdb/repl/drepl_fsmo.c @@ -91,11 +91,10 @@ NTSTATUS drepl_take_FSMO_role(struct irpc_message *msg, extended_op = DRSUAPI_EXOP_FSMO_REQ_PDC; break; default: - DEBUG(2,("Unknown role %u in role transfer\n", + DEBUG(0,("Unknown role %u in role transfer\n", (unsigned)role)); - r->out.result = WERR_DS_DRA_INTERNAL_ERROR; - talloc_free(tmp_ctx); - return NT_STATUS_OK; + /* IRPC messages are trusted, so this really should not happen */ + smb_panic("Unknown role despite dsdb_get_fsmo_role_info success"); } ret = samdb_dn_is_our_ntdsa(service->samdb, role_owner_dn, &is_us); |