[PATCH] Some cleanup patch for GUID_equal()
Volker Lendecke
Volker.Lendecke at SerNet.DE
Tue Jun 10 04:05:09 MDT 2014
Hi!
Review&push would be appreciated!
Thanks,
Volker
--
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de
-------------- next part --------------
From 80d12c55fb20057a2cbd88e60b9118b298611569 Mon Sep 17 00:00:00 2001
From: Volker Lendecke <vl at samba.org>
Date: Thu, 5 Jun 2014 09:56:55 +0000
Subject: [PATCH 1/2] libndr: Use GUID_compare in GUID_equal
Signed-off-by: Volker Lendecke <vl at samba.org>
---
librpc/ndr/uuid.c | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/librpc/ndr/uuid.c b/librpc/ndr/uuid.c
index cefe7b6..5558cb6 100644
--- a/librpc/ndr/uuid.c
+++ b/librpc/ndr/uuid.c
@@ -197,15 +197,7 @@ _PUBLIC_ bool GUID_all_zero(const struct GUID *u)
_PUBLIC_ bool GUID_equal(const struct GUID *u1, const struct GUID *u2)
{
- if (u1->time_low != u2->time_low ||
- u1->time_mid != u2->time_mid ||
- u1->time_hi_and_version != u2->time_hi_and_version ||
- u1->clock_seq[0] != u2->clock_seq[0] ||
- u1->clock_seq[1] != u2->clock_seq[1] ||
- memcmp(u1->node, u2->node, 6) != 0) {
- return false;
- }
- return true;
+ return (GUID_compare(u1, u2) == 0);
}
_PUBLIC_ int GUID_compare(const struct GUID *u1, const struct GUID *u2)
--
1.7.9.5
From 2e3b2aa3547b937aa5caf64f0b7c94f400975c53 Mon Sep 17 00:00:00 2001
From: Volker Lendecke <vl at samba.org>
Date: Thu, 5 Jun 2014 10:04:43 +0000
Subject: [PATCH 2/2] Use GUID_equal in a few places
Signed-off-by: Volker Lendecke <vl at samba.org>
---
source3/smbd/smb2_ioctl_network_fs.c | 2 +-
source4/dsdb/kcc/kcc_drs_replica_info.c | 4 ++--
source4/dsdb/kcc/kcc_periodic.c | 6 +++---
source4/dsdb/repl/drepl_notify.c | 4 ++--
source4/dsdb/repl/drepl_out_pull.c | 4 ++--
source4/dsdb/repl/drepl_partitions.c | 6 +++---
source4/dsdb/samdb/ldb_modules/util.c | 2 +-
source4/rpc_server/drsuapi/updaterefs.c | 7 ++++---
8 files changed, 18 insertions(+), 17 deletions(-)
diff --git a/source3/smbd/smb2_ioctl_network_fs.c b/source3/smbd/smb2_ioctl_network_fs.c
index 11843a4..5e0dc10 100644
--- a/source3/smbd/smb2_ioctl_network_fs.c
+++ b/source3/smbd/smb2_ioctl_network_fs.c
@@ -395,7 +395,7 @@ static NTSTATUS fsctl_validate_neg_info(TALLOC_CTX *mem_ctx,
}
}
- if (GUID_compare(&in_guid, &conn->smb2.client.guid) != 0) {
+ if (!GUID_equal(&in_guid, &conn->smb2.client.guid)) {
*disconnect = true;
return NT_STATUS_ACCESS_DENIED;
}
diff --git a/source4/dsdb/kcc/kcc_drs_replica_info.c b/source4/dsdb/kcc/kcc_drs_replica_info.c
index ac22312..52c36ca 100644
--- a/source4/dsdb/kcc/kcc_drs_replica_info.c
+++ b/source4/dsdb/kcc/kcc_drs_replica_info.c
@@ -629,8 +629,8 @@ static WERROR kccdrs_replica_get_info_neighbours(TALLOC_CTX *mem_ctx,
}
if (GUID_all_zero(&req_src_dsa_guid) ||
- GUID_compare(&req_src_dsa_guid, &reps_from->source_dsa_obj_guid) == 0)
- {
+ GUID_equal(&req_src_dsa_guid,
+ &reps_from->source_dsa_obj_guid)) {
if (i >= base_index) {
struct drsuapi_DsReplicaNeighbour neigh;
diff --git a/source4/dsdb/kcc/kcc_periodic.c b/source4/dsdb/kcc/kcc_periodic.c
index 4db4d4e..34bae96 100644
--- a/source4/dsdb/kcc/kcc_periodic.c
+++ b/source4/dsdb/kcc/kcc_periodic.c
@@ -42,8 +42,8 @@
*/
static bool kccsrv_same_source_dsa(struct repsFromToBlob *r1, struct repsFromToBlob *r2)
{
- return GUID_compare(&r1->ctr.ctr1.source_dsa_obj_guid,
- &r2->ctr.ctr1.source_dsa_obj_guid) == 0;
+ return GUID_equal(&r1->ctr.ctr1.source_dsa_obj_guid,
+ &r2->ctr.ctr1.source_dsa_obj_guid);
}
/*
@@ -498,7 +498,7 @@ NTSTATUS kccsrv_simple_update(struct kccsrv_service *s, TALLOC_CTX *mem_ctx)
struct GUID ntds_guid, invocation_id;
ntds_guid = samdb_result_guid(res->msgs[i], "objectGUID");
- if (GUID_compare(&ntds_guid, &s->ntds_guid) == 0) {
+ if (GUID_equal(&ntds_guid, &s->ntds_guid)) {
/* don't replicate with ourselves */
continue;
}
diff --git a/source4/dsdb/repl/drepl_notify.c b/source4/dsdb/repl/drepl_notify.c
index 905fe5f..3f2c851 100644
--- a/source4/dsdb/repl/drepl_notify.c
+++ b/source4/dsdb/repl/drepl_notify.c
@@ -255,14 +255,14 @@ static struct dreplsrv_partition_source_dsa *dreplsrv_find_notify_dsa(struct dre
/* first check the sources list */
for (s=p->sources; s; s=s->next) {
- if (GUID_compare(&s->repsFrom1->source_dsa_obj_guid, guid) == 0) {
+ if (GUID_equal(&s->repsFrom1->source_dsa_obj_guid, guid)) {
return s;
}
}
/* then the notifies list */
for (s=p->notifies; s; s=s->next) {
- if (GUID_compare(&s->repsFrom1->source_dsa_obj_guid, guid) == 0) {
+ if (GUID_equal(&s->repsFrom1->source_dsa_obj_guid, guid)) {
return s;
}
}
diff --git a/source4/dsdb/repl/drepl_out_pull.c b/source4/dsdb/repl/drepl_out_pull.c
index 58d8778..e64c91d 100644
--- a/source4/dsdb/repl/drepl_out_pull.c
+++ b/source4/dsdb/repl/drepl_out_pull.c
@@ -57,8 +57,8 @@ void drepl_reps_update(struct dreplsrv_service *s, const char *reps_attr,
}
for (i=0; i<count; i++) {
- if (GUID_compare(source_dsa_obj_guid,
- &reps[i].ctr.ctr1.source_dsa_obj_guid) == 0) {
+ if (GUID_equal(source_dsa_obj_guid,
+ &reps[i].ctr.ctr1.source_dsa_obj_guid)) {
break;
}
}
diff --git a/source4/dsdb/repl/drepl_partitions.c b/source4/dsdb/repl/drepl_partitions.c
index 7be069e..8c85ef6 100644
--- a/source4/dsdb/repl/drepl_partitions.c
+++ b/source4/dsdb/repl/drepl_partitions.c
@@ -364,7 +364,7 @@ static struct dreplsrv_partition_source_dsa *dreplsrv_find_source_dsa(struct dre
{
struct dreplsrv_partition_source_dsa *s;
for (s=list; s; s=s->next) {
- if (GUID_compare(&s->repsFrom1->source_dsa_obj_guid, guid) == 0) {
+ if (GUID_equal(&s->repsFrom1->source_dsa_obj_guid, guid)) {
return s;
}
}
@@ -415,8 +415,8 @@ static WERROR dreplsrv_partition_add_source_dsa(struct dreplsrv_service *s,
/* re-use an existing source if found */
for (s2=*listp; s2; s2=s2->next) {
- if (GUID_compare(&s2->repsFrom1->source_dsa_obj_guid,
- &source->repsFrom1->source_dsa_obj_guid) == 0) {
+ if (GUID_equal(&s2->repsFrom1->source_dsa_obj_guid,
+ &source->repsFrom1->source_dsa_obj_guid)) {
talloc_free(s2->repsFrom1->other_info);
*s2->repsFrom1 = *source->repsFrom1;
talloc_steal(s2, s2->repsFrom1->other_info);
diff --git a/source4/dsdb/samdb/ldb_modules/util.c b/source4/dsdb/samdb/ldb_modules/util.c
index 147e357..a71c49b 100644
--- a/source4/dsdb/samdb/ldb_modules/util.c
+++ b/source4/dsdb/samdb/ldb_modules/util.c
@@ -742,7 +742,7 @@ int dsdb_check_optional_feature(struct ldb_module *module, struct GUID op_featur
search_guid = samdb_result_guid(res->msgs[0], "msDS-OptionalFeatureGUID");
- if (GUID_compare(&search_guid, &op_feature_guid) == 0) {
+ if (GUID_equal(&search_guid, &op_feature_guid)) {
*feature_enabled = true;
break;
}
diff --git a/source4/rpc_server/drsuapi/updaterefs.c b/source4/rpc_server/drsuapi/updaterefs.c
index ae87117..6fdbf2e 100644
--- a/source4/rpc_server/drsuapi/updaterefs.c
+++ b/source4/rpc_server/drsuapi/updaterefs.c
@@ -50,8 +50,8 @@ static WERROR uref_add_dest(struct ldb_context *sam_ctx, TALLOC_CTX *mem_ctx,
}
for (i=0; i<reps.count; i++) {
- if (GUID_compare(&dest->source_dsa_obj_guid,
- &reps.r[i].ctr.ctr1.source_dsa_obj_guid) == 0) {
+ if (GUID_equal(&dest->source_dsa_obj_guid,
+ &reps.r[i].ctr.ctr1.source_dsa_obj_guid)) {
if (options & DRSUAPI_DRS_GETCHG_CHECK) {
return WERR_OK;
} else {
@@ -97,7 +97,8 @@ static WERROR uref_del_dest(struct ldb_context *sam_ctx, TALLOC_CTX *mem_ctx,
}
for (i=0; i<reps.count; i++) {
- if (GUID_compare(dest_guid, &reps.r[i].ctr.ctr1.source_dsa_obj_guid) == 0) {
+ if (GUID_equal(dest_guid,
+ &reps.r[i].ctr.ctr1.source_dsa_obj_guid)) {
if (i+1 < reps.count) {
memmove(&reps.r[i], &reps.r[i+1], sizeof(reps.r[i])*(reps.count-(i+1)));
}
--
1.7.9.5
More information about the samba-technical
mailing list