diff options
| author | Pavel Vomacka <pvomacka@redhat.com> | 2016-07-26 13:07:30 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2016-11-10 18:58:38 +0100 |
| commit | aa8a904c4a3953e799278de192d1613d21cde42a (patch) | |
| tree | 12072ae88815029969a693f0fba51071615c8d7c /install/ui | |
| parent | cd74f78ed74f8898c492024d0901cef9778df067 (diff) | |
| download | freeipa-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.js | 4 |
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()); }; |
