class-factory: Fix toString() for overloaded methods (#272)

frida · Apr 14, 2023 · 09c77f4 · 09c77f4
1 parent 4ac5c89
commit 09c77f4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/class-factory.js b/lib/class-factory.js
@@ -1409,6 +1409,10 @@ dispatcherPrototype = Object.create(Function.prototype, {
         return method.apply(receiver, args);
       }
 
+      if (this.methodName === 'toString') {
+        return `<class: ${receiver.$n}>`;
+      }
+
       throwOverloadError(this.methodName, this.overloads, 'argument types do not match any of:');
     }
   }

diff --git a/test/re/frida/MethodTest.java b/test/re/frida/MethodTest.java
@@ -506,6 +506,14 @@ public void performNowWorks() {
         assertEquals("Hello", script.getNextMessage());
     }
 
+    // Issue #271
+    @Test
+    public void toStringOnOverloadedToString() {
+        loadScript("const utf16 = Java.use('android.icu.text.UTF16');" +
+                "send(utf16.toString());");
+        assertEquals("<class: android.icu.text.UTF16>", script.getNextMessage());
+    }
+
     @Test
     public void replacementCanAcceptModifiedUTF8StringParameter() {
         loadScript("var Badger = Java.use('re.frida.Badger');" +