The branch, master has been updated via f1e4ae9 s4-drepl: Block GetNCChanges during a DsReplicaSync from 890137c s3: libsmb: Fix use-after-free when accessing pointer *p.
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit f1e4ae9971e61f7c4a43ae0049a1162cc2e3dafb Author: Andrew Bartlett <abart...@samba.org> Date: Sat Jul 22 22:00:59 2017 +1200 s4-drepl: Block GetNCChanges during a DsReplicaSync If we do not block these, we can get RPC faults (DCERPC_NCA_S_PROTO_ERROR) which gives WERR_WRITE_FAULT back to the DsReplicaSync call as there are two outstanding requests on the wire at the one time. We will get to the next operation as soon as this is finished when we call run_pending_ops(). BUG: https://bugzilla.samba.org/show_bug.cgi?id=12926 Signed-off-by: Andrew Bartlett <abart...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> Autobuild-User(master): Stefan Metzmacher <me...@samba.org> Autobuild-Date(master): Sun Jul 23 12:32:49 CEST 2017 on sn-devel-144 ----------------------------------------------------------------------- Summary of changes: source4/dsdb/repl/drepl_out_pull.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Changeset truncated at 500 lines: diff --git a/source4/dsdb/repl/drepl_out_pull.c b/source4/dsdb/repl/drepl_out_pull.c index 8af6412..8b8ecd9 100644 --- a/source4/dsdb/repl/drepl_out_pull.c +++ b/source4/dsdb/repl/drepl_out_pull.c @@ -198,7 +198,7 @@ void dreplsrv_run_pull_ops(struct dreplsrv_service *s) struct tevent_req *subreq; WERROR werr; - if (s->ops.current) { + if (s->ops.n_current || s->ops.current) { /* if there's still one running, we're done */ return; } -- Samba Shared Repository