diff options
| author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-08-08 04:44:09 +0200 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2016-09-09 09:31:34 +0000 |
| commit | 08f79e4686b97b8dae4acf0a804b8bbd1c29dd34 (patch) | |
| tree | ef38fd956f45eaf8716a7e05acb75dd55ef9d5dd | |
| parent | 518d3a6173742e0b8b8d30d8ad3584fe5af129c4 (diff) | |
it is possible that Sources is empty
See e.g.
http://crashreport.libreoffice.org/stats/crash_details/570429b8-21e3-494e-9677-ea95fa8a5293
Change-Id: I8c05efd61fa5a91511c06c660c49a0c470a96c88
Reviewed-on: https://gerrit.libreoffice.org/27947
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Reviewed-on: https://gerrit.libreoffice.org/28753
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
| -rw-r--r-- | extensions/source/update/check/updatecheck.cxx | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/extensions/source/update/check/updatecheck.cxx b/extensions/source/update/check/updatecheck.cxx index 8fe2363ab0e6..86dc5c5d0b27 100644 --- a/extensions/source/update/check/updatecheck.cxx +++ b/extensions/source/update/check/updatecheck.cxx @@ -846,6 +846,12 @@ UpdateCheck::download() State eState = m_eState; aGuard.clear(); + if (aInfo.Sources.empty()) + { + SAL_WARN("extension.updatecheck", "download called without source"); + return; + } + if( aInfo.Sources[0].IsDirect ) { // Ignore second click of a double click |