summaryrefslogtreecommitdiffstats
path: root/apps/user_ldap/js
diff options
context:
space:
mode:
authorStefan Weil <sw@weilnetz.de>2018-10-26 17:31:42 +0200
committerStefan Weil <sw@weilnetz.de>2018-10-26 17:31:42 +0200
commita226f65aa228652a7f0a35676055ce3f4323fdf8 (patch)
tree8545a206a9c9ecef046568aeed89da089e043a85 /apps/user_ldap/js
parent31c3814b0a3d33a25f4fbcc8777037f8c6c14533 (diff)
downloadnextcloud-server-a226f65aa228652a7f0a35676055ce3f4323fdf8.tar.gz
nextcloud-server-a226f65aa228652a7f0a35676055ce3f4323fdf8.zip
Fix off-by-one bugs
This fixes warnings from LGTM: Off-by-one index comparison against length may lead to out-of-bounds read. Signed-off-by: Stefan Weil <sw@weilnetz.de>
Diffstat (limited to 'apps/user_ldap/js')
-rw-r--r--apps/user_ldap/js/wizard/wizard.js4
1 files changed, 2 insertions, 2 deletions
diff --git a/apps/user_ldap/js/wizard/wizard.js b/apps/user_ldap/js/wizard/wizard.js
index 62a3fccbdfb..9ac03e117cd 100644
--- a/apps/user_ldap/js/wizard/wizard.js
+++ b/apps/user_ldap/js/wizard/wizard.js
@@ -45,7 +45,7 @@ OCA = OCA || {};
// for example, BaseDN detector needs the port. The port is typically found
// by the Port Detector. If BaseDN detector was run first, it will not have
// all necessary information. Only after Port Detector was executed…
- for (var i = 0; i <= detectors.length; i++) {
+ for (var i = 0; i < detectors.length; i++) {
model.registerDetector(detectors[i]);
}
@@ -61,7 +61,7 @@ OCA = OCA || {};
var view = new OCA.LDAP.Wizard.WizardView(model);
view.init();
view.setModel(model);
- for (var j = 0; j <= tabs.length; j++) {
+ for (var j = 0; j < tabs.length; j++) {
view.registerTab(tabs[j], '#ldapWizard' + (j + 2));
}