[Libreoffice-commits] core.git: Branch 'libreoffice-5-2' - desktop/source

[Markus Mohrhard]

 desktop/source/minidump/minidump_upload.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit 6c90ce201fb27a814434cd025748bfd16a8eb3ef
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Thu Jun 2 17:04:27 2016 +0200

    add some more error checking to minidump uploader
    
    Change-Id: Ia06fec80ea67e88b8c26cdeb5bfc9cee2459ad4a
    Reviewed-on: https://gerrit.libreoffice.org/25861
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    (cherry picked from commit f2d18f8a7125f4f4d1d72be11fb3fb6637f25fde)
    Reviewed-on: https://gerrit.libreoffice.org/25929
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/desktop/source/minidump/minidump_upload.cxx b/desktop/source/minidump/minidump_upload.cxx
index 3f17c04..30eaaaa 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.