summaryrefslogtreecommitdiffstats
path: root/install/ui
diff options
context:
space:
mode:
authorPavel Vomacka <pvomacka@redhat.com>2016-07-26 13:07:30 +0200
committerPetr Vobornik <pvoborni@redhat.com>2016-11-10 18:58:38 +0100
commitaa8a904c4a3953e799278de192d1613d21cde42a (patch)
tree12072ae88815029969a693f0fba51071615c8d7c /install/ui
parentcd74f78ed74f8898c492024d0901cef9778df067 (diff)
downloadfreeipa-aa8a904c4a3953e799278de192d1613d21cde42a.tar.gz
freeipa-aa8a904c4a3953e799278de192d1613d21cde42a.tar.xz
freeipa-aa8a904c4a3953e799278de192d1613d21cde42a.zip
Coverity - accessing attribute of variable which can point to null
Added check whether variable is pointing to null or not. Reviewed-By: Petr Vobornik <pvoborni@redhat.com>
Diffstat (limited to 'install/ui')
-rw-r--r--install/ui/src/freeipa/widget.js4
1 files changed, 2 insertions, 2 deletions
diff --git a/install/ui/src/freeipa/widget.js b/install/ui/src/freeipa/widget.js
index f0e95f523..041eaa20c 100644
--- a/install/ui/src/freeipa/widget.js
+++ b/install/ui/src/freeipa/widget.js
@@ -4939,7 +4939,7 @@ IPA.combobox_widget = function(spec) {
var value = that.list.val();
var option = $('option[value="'+value+'"]', that.list);
var next = option.next();
- if (!next.length) return;
+ if (!next || !next.length) return;
that.select(next.val());
};
@@ -4947,7 +4947,7 @@ IPA.combobox_widget = function(spec) {
var value = that.list.val();
var option = $('option[value="'+value+'"]', that.list);
var prev = option.prev();
- if (!prev.length) return;
+ if (!prev || !prev.length) return;
that.select(prev.val());
};