frida / frida-rust

Frida Rust bindings
Other
188 stars 53 forks source link

frida-gum: dereference listener objects #157

Closed ajwerner closed 2 months ago

ajwerner commented 2 months ago

When combined with https://github.com/frida/frida-rust/pull/153, fixes https://github.com/frida/frida-rust/issues/152.

s1341 commented 2 months ago

This means that you need to keep references to the returned Listeners alive for as long as you want the hooks to remain, right? That will break a lot of stuff.

ajwerner commented 2 months ago

No Frida increments the ref count when you attach and drops it when you detach. That’s why it was a leak.

s1341 commented 2 months ago

Ok. Let's see what CI says.

s1341 commented 2 months ago

Merged. Thanks!