Skip to content

frontend: Abort if obs-transitions failed to load - #13321

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
notr1ch:r1ch/transition-crash-fix
Aug 21, 2026
Merged

RytoEX merged 1 commit into
obsproject:masterfrom
notr1ch:r1ch/transition-crash-fix

Conversation

@notr1ch

@notr1ch notr1ch commented Apr 14, 2026

Copy link
Copy Markdown
Member

Description

The frontend UI assumes transitions are always available and crashes if none are present. Resolving the crashes resulted in a completely broken UI, so it's safe to assume that transitions are a core requirement for a working OBS.

Crash reports indicate that obs-transitions is failing to load for some users, so we should handle the case where no transitions load and abort with an error instead of crashing.

Motivation and Context

https://obsproject.com/logs/crashes/09jMYJwaWSRscFNd

How Has This Been Tested?

Deleted obs-transitions.dll, ran OBS.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have read the contributing document.
  • My code has been run through clang-format.
  • My code follows the project's style guidelines
  • My code is not on the master branch.
  • My code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@notr1ch notr1ch added the kind/bug Categorizes issue or PR as related to a bug. label Apr 14, 2026
Comment thread frontend/widgets/OBSBasic_Transitions.cpp
Comment thread frontend/widgets/OBSBasic_Transitions.cpp
The frontend UI assumes transitions are always available and crashes if
none are present. Resolving the crashes resulted in a completely broken
UI, so it's safe to assume that transitions are a core requirement for a
working OBS.

Crash reports indicate that obs-transitions is failing to load for some
users, so we should handle the case where no transitions load and abort
with an error instead of crashing.
@notr1ch
notr1ch force-pushed the r1ch/transition-crash-fix branch from 88cf21a to e943fd1 Compare May 5, 2026 21:55
@github-project-automation github-project-automation Bot moved this to Ready For Review in 33.0 Release Tracker Jul 6, 2026
@Warchamp7 Warchamp7 added this to the OBS Studio 33.0 milestone Jul 6, 2026
@Warchamp7 Warchamp7 moved this from Ready For Review to Ready For Merge in 33.0 Release Tracker Jul 6, 2026
@Warchamp7 Warchamp7 added the release-note/fix Groups pull request under the "Fixes" section in the associated patch notes label Aug 19, 2026
@RytoEX
RytoEX merged commit e86c7e5 into obsproject:master Aug 21, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Ready For Merge to Merged in 33.0 Release Tracker Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug. release-note/fix Groups pull request under the "Fixes" section in the associated patch notes

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants