Memory leak fix patch

Change-Id: I1e56111a23cf9c27a5cf047ae87790afb6a77fbe
Signed-off-by: brianjjones <brian.j.jones@intel.com>

diff --git a/packaging/phoned.changes b/packaging/phoned.changes
index 833e01f..df9a855 100644 (file)
--- a/packaging/phoned.changes
+++ b/packaging/phoned.changes
@@ -1,3 +1,6 @@
+* Mon Apr 14 2014 brianjjones <brian.j.jones@intel.com> accepted/tizen/ivi/release/20140404.212421@5627815
+- Memory leak fix patch
+
 * Fri Mar 07 2014 brianjjones <brian.j.jones@intel.com> c7ee743
 - Initial checkin for phone daemon for the HTML5 UI
 

diff --git a/src/obex.cpp b/src/obex.cpp
index 7be7846..7c0a966 100644 (file)
--- a/src/obex.cpp
+++ b/src/obex.cpp
@@ -808,7 +808,7 @@ void Obex::handleSignal(GDBusConnection       *connection,
     if(!strcmp(signal_name, "PropertiesChanged"))
     {
                char *objPath = NULL;
-               GVariantIter* iter, iter2;
+               GVariantIter* iter, *iter2;
 
                g_variant_get(parameters, "(sa{sv}as)", &objPath, &iter, &iter2);
 
@@ -839,12 +839,17 @@ void Obex::handleSignal(GDBusConnection       *connection,
                                                ctx->initiateNextSyncRequest();
                                        }
                                }
+                               g_free(prop);
+                               g_variant_unref(var);
                        }
                }
                else
                {
                        LoggerD("No objectPath found. Exiting.");
                }
+               g_variant_iter_free(iter);
+               g_variant_iter_free(iter2);
+               g_free(objPath);
     }
 }