diff options
| author | Pavel Vomacka <pvomacka@redhat.com> | 2016-07-25 17:44:56 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2016-11-10 18:58:38 +0100 |
| commit | 7be585dbb206ed12b25d09bfb2f5452ee9c125ae (patch) | |
| tree | 7b054e1042b6c6cede0421a90e9784739717dac4 /install/ui/src | |
| parent | ed74e14ab4a17c83cf6782e4b6fd41a2ce79594d (diff) | |
| download | freeipa-7be585dbb206ed12b25d09bfb2f5452ee9c125ae.tar.gz freeipa-7be585dbb206ed12b25d09bfb2f5452ee9c125ae.tar.xz freeipa-7be585dbb206ed12b25d09bfb2f5452ee9c125ae.zip | |
Coverity - true branch can't be executed
The 'data' 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 65bed7766..2eed6cc87 100644 --- a/install/ui/src/freeipa/rpc.js +++ b/install/ui/src/freeipa/rpc.js @@ -379,7 +379,8 @@ rpc.command = function(spec) { error_handler.call(this, xhr, text_status, /* error_thrown */ { name: text.get('@i18n:errors.http_error', 'HTTP Error')+' '+xhr.status, url: this.url, - message: data ? xhr.statusText : text.get('@i18n:errors.no_response', 'No response') + message: xhr ? xhr.statusText : + text.get('@i18n:errors.no_response', 'No response') }); } else if (IPA.version && data.version && IPA.version !== data.version) { |