diff options
author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-08-08 04:44:09 +0200 |
---|---|---|
committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-08-08 18:22:41 +0000 |
commit | 8cb19762323016f4bb31494404e5c7e29d4081a7 (patch) | |
tree | 3b1c4d237110da723d91979314934b10640b98e3 /extensions | |
parent | cdb708291b59ac89b43c24154f0edc77f237eadd (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>
Diffstat (limited to 'extensions')
-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 |