summaryrefslogtreecommitdiff
path: root/source4/dsdb/samdb
diff options
context:
space:
mode:
authorMatthieu Patou <mat@matws.net>2012-12-27 00:01:49 -0800
committerStefan Metzmacher <metze@samba.org>2013-01-17 17:10:32 +0100
commite1d50a6b24012b42f9987b439a4b94cd8db79999 (patch)
tree522de5ee1d39f9ec8e73c43981b854dfcea42080 /source4/dsdb/samdb
parenta0faf16ae9aefc4963b2583970509b1b23e27ce1 (diff)
downloadsamba-e1d50a6b24012b42f9987b439a4b94cd8db79999.tar.gz
samba-e1d50a6b24012b42f9987b439a4b94cd8db79999.tar.bz2
samba-e1d50a6b24012b42f9987b439a4b94cd8db79999.zip
dsdb-operational: Avoid doing the ldb_attr_cmp if bypass flag is not set
Most of the time this flag is not set and so we can avoid the strcasecmp in ldb_attr_cmp() Reviewed-by: Stefan Metzmacher <metze@samba.org> Autobuild-User(master): Stefan Metzmacher <metze@samba.org> Autobuild-Date(master): Thu Jan 17 17:10:32 CET 2013 on sn-devel-104
Diffstat (limited to 'source4/dsdb/samdb')
-rw-r--r--source4/dsdb/samdb/ldb_modules/operational.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/dsdb/samdb/ldb_modules/operational.c b/source4/dsdb/samdb/ldb_modules/operational.c
index c642ad8c92..b1122b3d42 100644
--- a/source4/dsdb/samdb/ldb_modules/operational.c
+++ b/source4/dsdb/samdb/ldb_modules/operational.c
@@ -612,7 +612,7 @@ struct op_controls_flags {
};
static bool check_keep_control_for_attribute(struct op_controls_flags* controls_flags, const char* attr) {
- if (ldb_attr_cmp(attr, "msDS-KeyVersionNumber") == 0 && controls_flags->bypassoperational) {
+ if (controls_flags->bypassoperational && ldb_attr_cmp(attr, "msDS-KeyVersionNumber") == 0 ) {
return true;
}
return false;