diff options
author | Matthieu Patou <mat@matws.net> | 2012-12-27 00:01:49 -0800 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2013-01-17 17:10:32 +0100 |
commit | e1d50a6b24012b42f9987b439a4b94cd8db79999 (patch) | |
tree | 522de5ee1d39f9ec8e73c43981b854dfcea42080 | |
parent | a0faf16ae9aefc4963b2583970509b1b23e27ce1 (diff) | |
download | samba-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
-rw-r--r-- | source4/dsdb/samdb/ldb_modules/operational.c | 2 |
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; |