[Libreoffice-commits] .: sd/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Fri Jan 25 17:57:31 PST 2013


 sd/source/ui/remotecontrol/BluetoothServer.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit e99b540d8985b87247519c951e6ef65a01b1f5c9
Author: Michael Meeks <michael.meeks at suse.com>
Date:   Sat Jan 26 12:32:17 2013 +1100

    sdremote: kill memory corruption / mis-matched alloc/free's.
    
    Change-Id: I253f28d7c83371282adde86c8eaadd6f25beff5a

diff --git a/sd/source/ui/remotecontrol/BluetoothServer.cxx b/sd/source/ui/remotecontrol/BluetoothServer.cxx
index beca8eb..b9bf825 100644
--- a/sd/source/ui/remotecontrol/BluetoothServer.cxx
+++ b/sd/source/ui/remotecontrol/BluetoothServer.cxx
@@ -127,7 +127,7 @@ bool BluetoothServer::isDiscoverable()
         return false;
     }
 
-    GHashTable* aProperties;
+    GHashTable* aProperties = NULL;
     aResult = dbus_g_proxy_call( aAdapter, "GetProperties", &aError,
                                 G_TYPE_INVALID,
                                 DBUS_TYPE_G_STRING_ANY_HASHTABLE, &aProperties,
@@ -144,7 +144,7 @@ bool BluetoothServer::isDiscoverable()
     gboolean aIsDiscoverable = g_value_get_boolean( (GValue*) g_hash_table_lookup(
                 aProperties, "Discoverable" ) );
 
-    g_free( aProperties );
+    g_hash_table_unref( aProperties );
     return aIsDiscoverable;
 #else // defined(LINUX) && defined(ENABLE_DBUS)
     return false;
@@ -196,7 +196,7 @@ void BluetoothServer::setDiscoverable( bool aDiscoverable )
     gboolean aPowered = g_value_get_boolean( (GValue*) g_hash_table_lookup(
                 aProperties, "Powered" ) );
 
-    g_free( aProperties );
+    g_hash_table_unref( aProperties );
     if ( !aPowered )
     {
         SAL_INFO( "sdremote.bluetooth", "Bluetooth adapter not powered, returning" );


More information about the Libreoffice-commits mailing list