Skip to content

Fix binary view event ownership#130

Merged
tinysec merged 1 commit into
masterfrom
fix/binary-view-event-ownership
Jul 22, 2026
Merged

Fix binary view event ownership#130
tinysec merged 1 commit into
masterfrom
fix/binary-view-event-ownership

Conversation

@tinysec

@tinysec tinysec commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • expose official BinaryViewType lifecycle event registration helpers
  • use explicit Cdecl callbacks with process-lifetime rooting
  • pass independently referenced BinaryView objects to managed callbacks
  • catch callback exceptions before they cross the native boundary
  • preserve Core.RegisterBinaryViewEvent as a compatibility forwarder

Verification

  • dotnet build BinaryNinja.csproj --no-restore
  • targeted real-core BinaryViewEventTests: 1/1 passed
  • full external real-core E2E: 257/257 passed

@tinysec
tinysec merged commit ed99278 into master Jul 22, 2026
1 check passed
@tinysec
tinysec deleted the fix/binary-view-event-ownership branch July 22, 2026 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant