[Libreoffice-commits] core.git: vcl/unx

[Caolán McNamara (via logerrit)]

 vcl/unx/gtk3/gtk3gtkframe.cxx |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit beabc0deddce2a5ce0a9f9b20316a7798a08318c
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Fri Mar 6 12:35:53 2020 +0000
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Mar 6 16:24:08 2020 +0100

    failure seen as setting length of -1 and returning null
    
    Change-Id: I84e7b3a4ad63e70499910f09bd4c70a43137fa10
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90103
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index a4f90290e3bd..36b12d9742e1 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -3456,8 +3456,12 @@ public:
             gint length(0);
             const guchar *rawdata = gtk_selection_data_get_data_with_length(m_pData,
                                                                             &length);
-            css::uno::Sequence<sal_Int8> aSeq(reinterpret_cast<const sal_Int8*>(rawdata), length);
-            aRet <<= aSeq;
+            // seen here was rawhide == nullptr and length set to -1
+            if (rawdata)
+            {
+                css::uno::Sequence<sal_Int8> aSeq(reinterpret_cast<const sal_Int8*>(rawdata), length);
+                aRet <<= aSeq;
+            }
         }
 
         gtk_selection_data_free(m_pData);