[SCM] Samba Shared Repository - branch v3-6-test updated

Jeremy Allison jra at samba.org
Tue Aug 17 13:03:07 MDT 2010


The branch, v3-6-test has been updated
       via  0060b1e... s3:winbindd: don't ignore 'result' in wb_dsgetdcname_done()
      from  ea41798... s3: Fix some error messages (cherry picked from commit ad871b73a26f3b48a522c8374c00aa5ff7d2eb4c)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-6-test


- Log -----------------------------------------------------------------
commit 0060b1ebac0960d95b5a24c7611d1f1568d29551
Author: Stefan Metzmacher <metze at samba.org>
Date:   Tue Aug 17 08:02:46 2010 +0200

    s3:winbindd: don't ignore 'result' in wb_dsgetdcname_done()
    
    Ignoring it could cause a segfault in winbindd_getdcname_recv()
    
    metze
    (cherry picked from commit aca7b22e96482bdd90e59973e25dc41690969bab)

-----------------------------------------------------------------------

Summary of changes:
 source3/winbindd/wb_dsgetdcname.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/winbindd/wb_dsgetdcname.c b/source3/winbindd/wb_dsgetdcname.c
index 994d14e..1334eb9 100644
--- a/source3/winbindd/wb_dsgetdcname.c
+++ b/source3/winbindd/wb_dsgetdcname.c
@@ -97,6 +97,10 @@ static void wb_dsgetdcname_done(struct tevent_req *subreq)
 		tevent_req_nterror(req, status);
 		return;
 	}
+	if (!NT_STATUS_IS_OK(result)) {
+		tevent_req_nterror(req, result);
+		return;
+	}
 	tevent_req_done(req);
 }
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list