[Libreoffice-commits] core.git: Branch 'libreoffice-5-4' - vcl/unx

Caolán McNamara caolanm at redhat.com
Sun Sep 24 22:15:01 UTC 2017


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

New commits:
commit 5deb6f29a4038f2de88abbc676e485f5bb5c150d
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/42721
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 7c880270eeaf..01ec0a15abd5 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