ViewFrame seems uninitialized when using UIContext.createTabForWidget

Open

#1,442 opened on Sep 11, 2019 7 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 4/5
Estimated time: 3-5 days
Newbie friendliness: 45/100

Issue type: Bug
Clarity: Mostly clear
Activity status: Stale
Tech stack: cpp
Domain: desktop

Research direction

Start with UIContext.createTabForWidget and trace how it creates a tab for an existing View such as LinearView or HexEditor. Reproduce the navigation-dialog crash in ViewFrame::getHistoryEntry and the graph-switch crash in ViewFrame::getTypeForView(QWidget*); done means these operations no longer crash for widgets added through this path.

Written by the indexing model from the issue text.

Description

Component: API Component: UI Effort: Medium Impact: Low

Multiple ViewFrame methods crash Binary Ninja when creating a new tab for an existing View widget like LinearView and HexEditor. I've found at least two methods crash, there's probably more. The two I've found to crash are ViewFrame::getHistoryEntry and ViewFrame::getTypeForView(QWidget*).

The first one occurs whenever the user tries to use the navigation dialog (g) to change location. Scrolling and clicking around works fine, but as soon as the dialog is used, it crashes on ViewFrame::getHistoryEntry().

The second one occured with a LinearView when I accidentally pressed the space bar and it tried to switch to the Graph view. That one crashed on ViewFrame::getTypeForView(QWidget*).

I suspect that the ViewFrame is not being created or initialized, which is causing null derefs when its member fields are accessed.

Dominant language: C++
Stars: 1.3k
Forks: 298
Avg merge: 5d 5h
Merged PRs (30d): 19

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

Eval bug: SIGSEGV in token-counting routes when the request arrives while the server is sleeping (stale vocab/mctx captured before the wake barrier) Open

Difficulty 2/5 1-3 hours Newbie friendliness 86/100

ggml-org/llama.cpp#29188 · Opened Sep 20, 2026
Sensor initialization takes very long when `--initial-sim-time` is set to current UNIX timestamp Open

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

gazebosim/gz-sensors#662 · 1 comment · Opened Sep 20, 2026
Aligned columns on the serial monitor while keeping CSV on the SD card Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 76/100

NorthernWidget/Margay_Library#31 · Opened Sep 20, 2026
v1 manifest list: existing_files_count is the table-wide total-data-files Open

comp-datalake

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

ClickHouse/ClickHouse#121222 · Opened Sep 20, 2026
Case collision between `Canvas` and `canvas` test directories causes platform-dependent source hashes Open

Difficulty 2/5 1-3 hours Newbie friendliness 68/100

LadybirdBrowser/ladybird#12123 · Opened Sep 20, 2026

More C++ issues