diff options
| author | Pavel Vomacka <pvomacka@redhat.com> | 2016-07-25 17:49:50 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2016-11-10 18:58:38 +0100 |
| commit | d94a2aa185defba38f2bbe2c5ee28f9b9defc0f2 (patch) | |
| tree | ea8d30bcf4dcee10ed1ef3b3a030e5958655693d /install/ui/src | |
| parent | 7be585dbb206ed12b25d09bfb2f5452ee9c125ae (diff) | |
| download | freeipa-d94a2aa185defba38f2bbe2c5ee28f9b9defc0f2.tar.gz freeipa-d94a2aa185defba38f2bbe2c5ee28f9b9defc0f2.tar.xz freeipa-d94a2aa185defba38f2bbe2c5ee28f9b9defc0f2.zip | |
Coverity - true branch can't be executed
The 'result' variable is always false because of previous condition.
Therefore there is direct assignment.
Reviewed-By: Petr Vobornik <pvoborni@redhat.com>
Diffstat (limited to 'install/ui/src')
| -rw-r--r-- | install/ui/src/freeipa/rpc.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/install/ui/src/freeipa/rpc.js b/install/ui/src/freeipa/rpc.js index 2eed6cc87..f3cc48cd4 100644 --- a/install/ui/src/freeipa/rpc.js +++ b/install/ui/src/freeipa/rpc.js @@ -636,7 +636,8 @@ rpc.batch_command = function(spec) { if (!result) { name = text.get('@i18n:errors.internal_error', 'Internal Error')+' '+xhr.status; - message = result ? xhr.statusText : text.get('@i18n:errors.internal_error', 'Internal Error'); + message = xhr ? xhr.statusText : + text.get('@i18n:errors.internal_error', 'Internal Error'); that.errors.add(command, name, message, text_status); |