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

Karolin Seeger kseeger at samba.org
Thu May 31 12:22:18 MDT 2012


The branch, v3-6-test has been updated
       via  6797759 Fix bad bugfix for bug #8910 - resolve_ads() code can return zero addresses and miss valid DC IP addresses
      from  f68121d Fix bug #8882 - Broken processing of %U with vfs_full_audit when force user is set.

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


- Log -----------------------------------------------------------------
commit 6797759a36b50f82fb1f75a4cc9c2555a79073a2
Author: Ira Cooper <ira at wakeful.net>
Date:   Wed May 30 11:50:06 2012 -0700

    Fix bad bugfix for bug #8910 - resolve_ads() code can return zero addresses and miss valid DC IP addresses
    
    Original code incorrectly used a while() instead of a for() loop.
    We need to iterate over the entire array here.
    
    Autobuild-User: Jeremy Allison <jra at samba.org>
    Autobuild-Date: Wed May 30 23:29:03 CEST 2012 on sn-devel-104
    (cherry picked from commit fc7e1113c34819de6fc8053fb81ee2e0e970bec2)

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

Summary of changes:
 source3/libsmb/namequery.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/libsmb/namequery.c b/source3/libsmb/namequery.c
index d0ab62f..4ba7204 100644
--- a/source3/libsmb/namequery.c
+++ b/source3/libsmb/namequery.c
@@ -1934,7 +1934,7 @@ static NTSTATUS resolve_ads(const char *name,
 
 	*return_count = 0;
 
-	while ( i < numdcs && (*return_count<numaddrs) ) {
+	for (i = 0; i < numdcs && (*return_count<numaddrs); i++ ) {
 		/* If we don't have an IP list for a name, lookup it up */
 		if (!dcs[i].ss_s) {
 			/* We need to get all IP addresses here. */


-- 
Samba Shared Repository


More information about the samba-cvs mailing list