[SCM] SAMBA-CTDB repository - branch v3-2-ctdb updated - build_3.2.3_ctdb.50-63-g7b3bcfa

Michael Adam obnox at samba.org
Mon Nov 24 14:34:07 GMT 2008


The branch, v3-2-ctdb has been updated
       via  7b3bcfa6caab6d46a8c4b21b8d05404510753e94 (commit)
      from  8a918ee94df9a59c145a66950de4dcec3c0fd523 (commit)

http://gitweb.samba.org/?p=obnox/samba-ctdb.git;a=shortlog;h=v3-2-ctdb


- Log -----------------------------------------------------------------
commit 7b3bcfa6caab6d46a8c4b21b8d05404510753e94
Author: Stefan Metzmacher <metze at samba.org>
Date:   Mon Nov 24 15:07:56 2008 +0100

    libads/ldap.c: return an error instead of crashing when no realm is given
    
    The bug was triggered by "net ads info -S 127.8.7.6" (where 127.8.7.6 doesn't exist)
    and "disable netbios = yes".
    
    metze
    
    Signed-off-by: Michael Adam <obnox at samba.org>
    (cherry picked from commit e2a60d2e1fd7455f08e73f8d1c513ca240c0f0af)
    (cherry picked from commit 8fc213fcd4c5a6ddaa0d44965bcd0ca1afbbbc84)

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

Summary of changes:
 source/libads/ldap.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/libads/ldap.c b/source/libads/ldap.c
index 842e4fd..1a45aff 100644
--- a/source/libads/ldap.c
+++ b/source/libads/ldap.c
@@ -301,11 +301,11 @@ static NTSTATUS ads_find_dc(ADS_STRUCT *ads)
 			if ( use_own_domain )
 				c_realm = lp_workgroup();
 		}
+	}
 
-		if ( !c_realm || !*c_realm ) {
-			DEBUG(0,("ads_find_dc: no realm or workgroup!  Don't know what to do\n"));
-			return NT_STATUS_INVALID_PARAMETER; /* rather need MISSING_PARAMETER ... */
-		}
+	if ( !c_realm || !*c_realm ) {
+		DEBUG(0,("ads_find_dc: no realm or workgroup!  Don't know what to do\n"));
+		return NT_STATUS_INVALID_PARAMETER; /* rather need MISSING_PARAMETER ... */
 	}
 
 	realm = c_realm;


-- 
SAMBA-CTDB repository


More information about the samba-cvs mailing list