changeset 591:2b2073cc9a19

Fix 2 memory leaks, pass corresponding C++ unit tests
author Adam Domurad <adomurad@redhat.com>
date Thu, 20 Dec 2012 11:11:05 -0500
parents 6251ec285570
children 95fc28e972ad
files ChangeLog plugin/icedteanp/IcedTeaScriptablePluginObject.cc
diffstat 2 files changed, 8 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Thu Dec 20 11:28:28 2012 -0500
+++ b/ChangeLog	Thu Dec 20 11:11:05 2012 -0500
@@ -1,3 +1,9 @@
+2012-12-20  Adam Domurad  <adomurad@redhat.com>
+
+	* plugin/icedteanp/IcedTeaScriptablePluginObject.cc
+	(IcedTeaScriptableJavaObject::deAllocate): Fix memory leak
+	(IcedTeaScriptableJavaPackageObject::deAllocate): Fix memory leak
+
 2012-12-20  Adam Domurad  <adomurad@redhat.com>
 
 	* tests/cpp-unit-tests/browser_mock.cc
--- a/plugin/icedteanp/IcedTeaScriptablePluginObject.cc	Thu Dec 20 11:28:28 2012 -0500
+++ b/plugin/icedteanp/IcedTeaScriptablePluginObject.cc	Thu Dec 20 11:11:05 2012 -0500
@@ -197,7 +197,7 @@
 void
 IcedTeaScriptableJavaPackageObject::deAllocate(NPObject *npobj)
 {
-    browser_functions.releaseobject(npobj);
+    delete (IcedTeaScriptableJavaPackageObject*)npobj;
 }
 
 void
@@ -489,7 +489,7 @@
 void
 IcedTeaScriptableJavaObject::deAllocate(NPObject *npobj)
 {
-	browser_functions.releaseobject(npobj);
+    delete (IcedTeaScriptableJavaObject*)npobj;
 }
 
 void