Unreal Engine Issues

UE-137086

GetStaticMeshViewport() is called without checking if it returns a valid viewport causing the Editor to crash

GetStaticMeshViewport() is called without checking if it returns a valid viewport causing the Editor to crash

Description

This is not a regression.
Tested in:
//UE4/Release-4.27 CL18319896 Binary
//UE5/Release-5.0-EarlyAccess CL16682836 Binary
//UE5/Release-5.0 CL18407655 Personal P4

When opening a static mesh without a valid viewport the editor crashes.

Steps to Reproduce
  1. Open any static mesh in a static mesh editor
  2. Close the preview viewport tab in the static mesh editor
    • Click the tiny Blue(UE5) or Yellow(UE4) triangle in the top left corner of the viewport
  3. Close the static mesh editor
  4. Open any static mesh in the static mesh editor

Expected Results:
Static Mesh Editor reopens without the viewport.

Actual Results:
Editor crashes.

Callstack
UnrealEditor-StaticMeshEditor!FStaticMeshEditor::ExtendToolBar() [StaticMeshEditor.cpp:911]
UnrealEditor-StaticMeshEditor!FStaticMeshEditor::InitStaticMeshEditor(EToolkitMode::Type,TSharedPtr<IToolkitHost,0> const &,UStaticMesh *) [StaticMeshEditor.cpp:266]
UnrealEditor-StaticMeshEditor!FStaticMeshEditorModule::CreateStaticMeshEditor(EToolkitMode::Type,TSharedPtr<IToolkitHost,0> const &,UStaticMesh *) [StaticMeshEditorModule.cpp:51]
UnrealEditor-AssetTools!FAssetTypeActions_StaticMesh::OpenAssetEditor(TArray<UObject *,TSizedDefaultAllocator<32> > const &,TSharedPtr<IToolkitHost,0>) [AssetTypeActions_StaticMesh.cpp:84]
UnrealEditor-UnrealEd!7ffca11e0000  + ced2da
UnrealEditor-UnrealEd!7ffca11e0000  + cedbb0
UnrealEditor-UnrealEd!7ffca11e0000  + ced95a
UnrealEditor-ContentBrowserAssetDataSource!ContentBrowserAssetData::EditOrPreviewAssetFileItems(TArrayView<TSharedRef<FContentBrowserAssetFileItemDataPayload const ,0> const ,int>,bool) [ContentBrowserAssetDataCore.cpp:287]
UnrealEditor-ContentBrowserAssetDataSource!ContentBrowserAssetData::EditOrPreviewItems(IAssetTools *,UContentBrowserDataSource const *,TArrayView<FContentBrowserItemData const ,int>,bool) [ContentBrowserAssetDataCore.cpp:307]
UnrealEditor-ContentBrowserAssetDataSource!UContentBrowserAssetDataSource::BulkEditItems(TArrayView<FContentBrowserItemData const ,int>) [ContentBrowserAssetDataSource.cpp:1046]
UnrealEditor-ContentBrowser!SContentBrowser::OnItemsActivated(TArrayView<FContentBrowserItem const ,int>,EAssetTypeActivationMethod::Type) [SContentBrowser.cpp:2454]
UnrealEditor-ContentBrowser!TBaseSPMethodDelegateInstance<0,SContentBrowser,0,void ,FDefaultDelegateUserPolicy>::Execute(TArrayView<FContentBrowserItem const ,int>,EAssetTypeActivationMethod::Type) [DelegateInstancesImpl.h:290]
UnrealEditor-ContentBrowser!SAssetView::OnListMouseButtonDoubleClick(TSharedPtr<FAssetViewItem,0>) [SAssetView.cpp:3560]
UnrealEditor-ContentBrowser!TBaseSPMethodDelegateInstance<0,SAssetView,0,void ,FDefaultDelegateUserPolicy>::ExecuteIfSafe(TSharedPtr<FAssetViewItem,0>) [DelegateInstancesImpl.h:307]
UnrealEditor-ContentBrowser!TDelegate<void ,FDefaultDelegateUserPolicy>::ExecuteIfBound<void,0>(TSharedPtr<FAssetViewItem,0>) [DelegateSignatureImpl.inl:599]
UnrealEditor-ContentBrowser!SListView<TSharedPtr<FAssetViewItem,0> >::Private_OnItemDoubleClicked(TSharedPtr<FAssetViewItem,0>) [SListView.h:1080]
UnrealEditor-ContentBrowser!STableRow<TSharedPtr<FAssetViewItem,0> >::OnMouseButtonDoubleClick(FGeometry const &,FPointerEvent const &) [STableRow.h:461]
UnrealEditor-Slate!static class FReply FEventRouter::Route<FReply,FEventRouter::FBubblePolicy,FPointerEvent,<lambda_38816398cd24931ec4d21c3bfc532a3b> >(class FSlateApplication *, class FEventRouter::FBubblePolicy, struct FPointerEvent, const class FSlateApplication::RoutePointerDoubleClickEvent::__l2::<lambda_38816398cd24931ec4d21c3bfc532a3b> & const, ESlateDebuggingInputEvent) [SlateApplication.cpp:380]
UnrealEditor-Slate!FSlateApplication::RoutePointerDoubleClickEvent(FWidgetPath const &,FPointerEvent const &) [SlateApplication.cpp:5302]
UnrealEditor-Slate!FSlateApplication::ProcessMouseButtonDoubleClickEvent(TSharedPtr<FGenericWindow,0> const &,FPointerEvent const &) [SlateApplication.cpp:5289]
UnrealEditor-Slate!FSlateApplication::OnMouseDoubleClick(TSharedPtr<FGenericWindow,0> const &,EMouseButtons::Type,FVector2D) [SlateApplication.cpp:5252]
UnrealEditor-ApplicationCore!FWindowsApplication::ProcessDeferredMessage(FDeferredWindowsMessage const &) [WindowsApplication.cpp:2200]
UnrealEditor-ApplicationCore!FWindowsApplication::DeferMessage(TSharedPtr<FWindowsWindow,0> &,HWND__ *,unsigned int,unsigned __int64,__int64,int,int,unsigned int) [WindowsApplication.cpp:2689]
UnrealEditor-ApplicationCore!FWindowsApplication::ProcessMessage(HWND__ *,unsigned int,unsigned __int64,__int64) [WindowsApplication.cpp:1872]
UnrealEditor-ApplicationCore!FWindowsApplication::AppWndProc(HWND__ *,unsigned int,unsigned __int64,__int64) [WindowsApplication.cpp:896]
USER32!7ffd11690000  + 163ed
USER32!7ffd11690000  + 15de2
UnrealEditor-ApplicationCore!FWindowsPlatformApplicationMisc::PumpMessages(bool) [WindowsPlatformApplicationMisc.cpp:131]
UnrealEditor!7ff6938b0000  + 76cb
UnrealEditor!7ff6938b0000  + 20b2c
UnrealEditor!7ff6938b0000  + 20c0a
UnrealEditor!7ff6938b0000  + 3573d
UnrealEditor!7ff6938b0000  + 386ba
KERNEL32!7ffd11e00000  + 17bd4
ntdll!7ffd12420000  + 6ced1

Have Comments or More Details?

There's no existing public thread on this issue, so head over to Questions & Answers just mention UE-137086 in the post.

0
Login to Vote
Fixed
ComponentUE - Editor
Affects Versions5.0
Target Fix5.0
Fix Commit18427012
Main Commit19203395
Release Commit18427012
CreatedDec 9, 2021
ResolvedDec 9, 2021
UpdatedJan 26, 2023
View Jira Issue