diff options
author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-06-02 17:04:27 +0200 |
---|---|---|
committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-06-05 21:22:18 +0000 |
commit | 6c90ce201fb27a814434cd025748bfd16a8eb3ef (patch) | |
tree | aa32464f72277d1cd2024d08d2b6834d8ebf8393 /desktop | |
parent | 22d4a2f8a6dbbd15ad7236a6c094804e0211bdad (diff) |
add some more error checking to minidump uploader
Change-Id: Ia06fec80ea67e88b8c26cdeb5bfc9cee2459ad4a
Reviewed-on: https://gerrit.libreoffice.org/25861
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
(cherry picked from commit f2d18f8a7125f4f4d1d72be11fb3fb6637f25fde)
Reviewed-on: https://gerrit.libreoffice.org/25929
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'desktop')
-rw-r--r-- | desktop/source/minidump/minidump_upload.cxx | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/desktop/source/minidump/minidump_upload.cxx b/desktop/source/minidump/minidump_upload.cxx index 3f17c04c0474..30eaaaa0d39c 100644 --- a/desktop/source/minidump/minidump_upload.cxx +++ b/desktop/source/minidump/minidump_upload.cxx @@ -100,6 +100,12 @@ bool uploadContent(std::map<std::string, std::string>& parameters) if (url.empty()) return false; + if (file.empty()) + return false; + + if (version.empty()) + return false; + curl_easy_setopt(curl, CURLOPT_URL, url.c_str()); curl_easy_setopt(curl, CURLOPT_USERAGENT, kUserAgent); // Set proxy information if necessary. |