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

Caolán McNamara caolanm at redhat.com
Sun Sep 24 20:16:34 UTC 2017


 vcl/unx/gtk3/gtk3gtkinst.cxx |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit 4aa12018c8078cabf1b74e5564cfac8b8a53b787
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Sun Sep 24 20:35:51 2017 +0100

    fix some leaks
    
    Change-Id: I06a3971d1d269b49b2a7954f977469fbc3d16f35
    Reviewed-on: https://gerrit.libreoffice.org/42719
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 3cab164d80cf..cda9252f1dd1 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -106,7 +106,10 @@ std::vector<css::datatransfer::DataFlavor> GtkTransferable::getTransferDataFlavo
 
         // omit text/plain;charset=unicode since it is not well defined
         if (rtl_str_compare(pName, "text/plain;charset=unicode") == 0)
+        {
+            g_free(pName);
             continue;
+        }
 
         for (size_t j = 0; j < SAL_N_ELEMENTS(aConversionTab); ++j)
         {
@@ -122,6 +125,7 @@ std::vector<css::datatransfer::DataFlavor> GtkTransferable::getTransferDataFlavo
         // them out for now before they confuse this code's clients:
         if (rtl_str_indexOfChar(pFinalName, '/') == -1)
         {
+            g_free(pName);
             continue;
         }
 


More information about the Libreoffice-commits mailing list