[SCM] Samba Shared Repository - branch master updated

Matthias Dieter Wallnöfer mdw at samba.org
Tue Mar 16 01:54:40 MDT 2010


The branch, master has been updated
       via  559575f... s4:dsdb/kcc/*.c - fix up wrong typed counters
      from  2a04aa0... s4-drs: Torture testcase for DS_REPL_INFO_METADATA_2_FOR_OBJ infoType of DsGetReplInfo()

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 559575fe40c2d0d860868276c42efacb46dc59aa
Author: Matthias Dieter Wallnöfer <mwallnoefer at yahoo.de>
Date:   Tue Mar 16 08:49:22 2010 +0100

    s4:dsdb/kcc/*.c - fix up wrong typed counters

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

Summary of changes:
 source4/dsdb/kcc/kcc_connection.c       |    6 ++++--
 source4/dsdb/kcc/kcc_deleted.c          |    2 +-
 source4/dsdb/kcc/kcc_drs_replica_info.c |   20 ++++++++++----------
 source4/dsdb/kcc/kcc_periodic.c         |    4 ++--
 source4/dsdb/kcc/kcc_topology.c         |   10 +++++-----
 5 files changed, 22 insertions(+), 20 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/dsdb/kcc/kcc_connection.c b/source4/dsdb/kcc/kcc_connection.c
index e78a615..e8f2e59 100644
--- a/source4/dsdb/kcc/kcc_connection.c
+++ b/source4/dsdb/kcc/kcc_connection.c
@@ -129,7 +129,8 @@ void kccsrv_apply_connections(struct kccsrv_service *s,
 			      struct kcc_connection_list *ntds_list,
 			      struct kcc_connection_list *dsa_list)
 {
-	int i, j, deleted = 0, added = 0, ret;
+	unsigned int i, j, deleted = 0, added = 0;
+	int ret;
 
 	for (i = 0; ntds_list && i < ntds_list->count; i++) {
 		struct kcc_connection *ntds = &ntds_list->servers[i];
@@ -169,7 +170,8 @@ void kccsrv_apply_connections(struct kccsrv_service *s,
 struct kcc_connection_list *kccsrv_find_connections(struct kccsrv_service *s,
 						    TALLOC_CTX *mem_ctx)
 {
-	int ret, i;
+	unsigned int i;
+	int ret;
 	struct ldb_dn *base_dn;
 	struct ldb_result *res;
 	const char *attrs[] = { "objectGUID", "fromServer", NULL };
diff --git a/source4/dsdb/kcc/kcc_deleted.c b/source4/dsdb/kcc/kcc_deleted.c
index db47333..dae9a8e 100644
--- a/source4/dsdb/kcc/kcc_deleted.c
+++ b/source4/dsdb/kcc/kcc_deleted.c
@@ -62,7 +62,7 @@ NTSTATUS kccsrv_check_deleted(struct kccsrv_service *s, TALLOC_CTX *mem_ctx)
 		struct ldb_dn *do_dn;
 		struct ldb_result *res;
 		const char *attrs[] = { "whenChanged", NULL };
-		int i;
+		unsigned int i;
 
 		ret = dsdb_get_deleted_objects_dn(s->samdb, mem_ctx, part->dn, &do_dn);
 		if (ret != LDB_SUCCESS) {
diff --git a/source4/dsdb/kcc/kcc_drs_replica_info.c b/source4/dsdb/kcc/kcc_drs_replica_info.c
index 98defc8..c64753b 100644
--- a/source4/dsdb/kcc/kcc_drs_replica_info.c
+++ b/source4/dsdb/kcc/kcc_drs_replica_info.c
@@ -175,7 +175,7 @@ static WERROR kccdrs_replica_get_info_obj_metadata2(TALLOC_CTX *mem_ctx,
 						    struct drsuapi_DsReplicaGetInfo *r,
 						    union drsuapi_DsReplicaInfo *reply,
 						    struct ldb_dn *dn,
-						    int base_index)
+						    uint32_t base_index)
 {
 	WERROR status;
 	struct replPropertyMetaDataBlob omd_ctr;
@@ -383,7 +383,7 @@ static WERROR get_master_ncs(TALLOC_CTX *mem_ctx, struct ldb_context *samdb,
 	struct ncList *nc_list = NULL;
 	struct ncList *nc_list_elem;
 	int ret;
-	int i;
+	unsigned int i;
 	char *nc_str;
 
 	ret = ldb_search(samdb, mem_ctx, &res, ldb_get_config_basedn(samdb),
@@ -401,7 +401,7 @@ static WERROR get_master_ncs(TALLOC_CTX *mem_ctx, struct ldb_context *samdb,
 
 	for (i = 0; i < res->count; i++) {
 		struct ldb_message_element *msg_elem = ldb_msg_find_element(res->msgs[i], "hasMasterNCs");
-		int k;
+		unsigned int k;
 
 		if (!msg_elem || msg_elem->num_values == 0) {
 			DEBUG(0,(__location__ ": Failed: Attribute hasMasterNCs not found - %s\n",
@@ -563,18 +563,18 @@ static WERROR kccdrs_replica_get_info_neighbours(TALLOC_CTX *mem_ctx,
 						 struct ldb_context *samdb,
 						 struct drsuapi_DsReplicaGetInfo *r,
 						 union drsuapi_DsReplicaInfo *reply,
-						 int base_index,
+						 uint32_t base_index,
 						 struct GUID req_src_dsa_guid,
 						 const char *object_dn_str)
 {
 	WERROR status;
-	int i, j;
+	uint32_t i, j;
 	struct ldb_dn *nc_dn = NULL;
 	struct ncList *p_nc_list = NULL;
 	struct repsFromToBlob *reps_from_blob = NULL;
 	struct repsFromTo2 *reps_from = NULL;
 	uint32_t c_reps_from;
-	int i_rep;
+	uint32_t i_rep;
 	struct drsuapi_DsReplicaNeighbour neigh;
 	struct ncList *nc_list = NULL;
 
@@ -680,18 +680,18 @@ static WERROR kccdrs_replica_get_info_repsto(TALLOC_CTX *mem_ctx,
 					     struct ldb_context *samdb,
 					     struct drsuapi_DsReplicaGetInfo *r,
 					     union drsuapi_DsReplicaInfo *reply,
-					     int base_index,
+					     uint32_t base_index,
 					     struct GUID req_src_dsa_guid,
 					     const char *object_dn_str)
 {
 	WERROR status;
-	int i, j;
+	uint32_t i, j;
 	struct ncList *p_nc_list = NULL;
 	struct ldb_dn *nc_dn = NULL;
 	struct repsFromToBlob *reps_to_blob;
 	struct repsFromTo2 *reps_to;
 	uint32_t c_reps_to;
-	int i_rep;
+	uint32_t i_rep;
 	struct drsuapi_DsReplicaNeighbour neigh;
 	struct ncList *nc_list = NULL;
 
@@ -754,7 +754,7 @@ NTSTATUS kccdrs_replica_get_info(struct irpc_message *msg,
 	WERROR status;
 	struct drsuapi_DsReplicaGetInfoRequest1 *req1;
 	struct drsuapi_DsReplicaGetInfoRequest2 *req2;
-	int base_index;
+	uint32_t base_index;
 	union drsuapi_DsReplicaInfo *reply;
 	struct GUID req_src_dsa_guid;
 	const char *object_dn_str = NULL;
diff --git a/source4/dsdb/kcc/kcc_periodic.c b/source4/dsdb/kcc/kcc_periodic.c
index d27e922..14e15e8 100644
--- a/source4/dsdb/kcc/kcc_periodic.c
+++ b/source4/dsdb/kcc/kcc_periodic.c
@@ -40,7 +40,7 @@
  */
 static bool reps_in_list(struct repsFromToBlob *r, struct repsFromToBlob *reps, uint32_t count)
 {
-	unsigned int i;
+	uint32_t i;
 	for (i=0; i<count; i++) {
 		if (strcmp(r->ctr.ctr1.other_info->dns_name, 
 			   reps[i].ctr.ctr1.other_info->dns_name) == 0 &&
@@ -66,7 +66,7 @@ static NTSTATUS kccsrv_add_repsFrom(struct kccsrv_service *s, TALLOC_CTX *mem_ct
 		struct repsFromToBlob *old_reps;
 		uint32_t old_count;
 		WERROR werr;
-		unsigned int i;
+		uint32_t i;
 		bool modified = false;
 
 		werr = dsdb_loadreps(s->samdb, mem_ctx, p->dn, "repsFrom", &old_reps, &old_count);
diff --git a/source4/dsdb/kcc/kcc_topology.c b/source4/dsdb/kcc/kcc_topology.c
index a2dd4dd..3530d0e 100644
--- a/source4/dsdb/kcc/kcc_topology.c
+++ b/source4/dsdb/kcc/kcc_topology.c
@@ -791,7 +791,7 @@ static NTSTATUS kcctpl_setup_graph(struct ldb_context *ldb, TALLOC_CTX *mem_ctx,
 							NULL };
 	int ret;
 	struct GUID_list vertex_ids;
-	uint32_t i;
+	unsigned int i;
 	NTSTATUS status;
 	struct ldb_message *site;
 	uint64_t site_opts;
@@ -1075,7 +1075,7 @@ static NTSTATUS kcctpl_get_all_bridgehead_dcs(struct ldb_context *ldb,
 	struct ldb_message *site, *schema;
 	const char * const dc_attrs[] = { "objectGUID", "options", NULL };
 	struct ldb_message_element *el;
-	uint32_t i;
+	unsigned int i;
 	bool rodc;
 	const char *transport_name, *transport_address_attr;
 	uint64_t site_opts;
@@ -1454,7 +1454,7 @@ static NTSTATUS kcctpl_color_vertices(struct ldb_context *ldb,
 		const char * const attrs[] = { "objectGUID", "name",
 					       "transportAddressAttribute",
 					       NULL };
-		uint32_t j;
+		unsigned int j;
 
 		vertex = &graph->vertices.data[i];
 
@@ -2730,7 +2730,7 @@ static NTSTATUS kcctpl_create_connection(struct ldb_context *ldb,
 	const char * const attrs[] = { "objectGUID", "parent", "fromServer",
 				       "transportType", "schedule", "options",
 				       "enabledConnection", NULL };
-	uint32_t i, valid_connections;
+	unsigned int i, valid_connections;
 	struct GUID_list keep_connections;
 
 	tmp_ctx = talloc_new(ldb);
@@ -3369,7 +3369,7 @@ static NTSTATUS kcctpl_create_intersite_connections(struct ldb_context *ldb,
 	const char * const attrs[] = { "enabled", "systemFlags", "nCName",
 					NULL };
 	int ret;
-	uint32_t i;
+	unsigned int i;
 
 	all_connected = true;
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list