Fix many memory leaks due to utf8fromidentifier misuse

1 files changed, 11 insertions, 1 deletions

diff --git a/tests/cpp-unit-tests/browser_mock.cc b/tests/cpp-unit-tests/browser_mock.cc
index e333be7..8724687 100644
--- a/tests/cpp-unit-tests/browser_mock.cc
+++ b/tests/cpp-unit-tests/browser_mock.cc
@@ -90,6 +90,15 @@ static NPObject* mock_createobject(NPP instance, NPClass* np_class) {
 	return obj;
 }
 
+static NPUTF8* mock_utf8fromidentifier(NPIdentifier id) {
+    // Treat NPIdentifier (== void pointer) as a pointer to characters for simplicity
+    const NPUTF8* str = (const NPUTF8*) id;
+    // We expect this string to be freed with 'memfree'
+    NPUTF8* copy = (NPUTF8*) mock_memalloc(strlen(str) + 1);
+    memcpy(copy, str, strlen(str) + 1);
+    return copy;
+}
+
 void browsermock_setup_functions() {
     memset(&browser_functions, 0, sizeof(NPNetscapeFuncs));
 
@@ -98,7 +107,8 @@ void browsermock_setup_functions() {
 
     browser_functions.createobject = &mock_createobject;
     browser_functions.retainobject = &mock_retainobject;
-    browser_functions.releaseobject= &mock_releaseobject;
+    browser_functions.releaseobject = &mock_releaseobject;
+    browser_functions.utf8fromidentifier = &mock_utf8fromidentifier;
 }
 
 void browsermock_clear_state() {