diff options
| author | Andrey Kleshchev <andreykproductengine@lindenlab.com> | 2024-01-30 01:50:15 +0200 |
|---|---|---|
| committer | Andrey Kleshchev <117672381+akleshchev@users.noreply.github.com> | 2024-01-30 17:54:37 +0200 |
| commit | d3448fa204a648d61d07f12ecc982841160380d2 (patch) | |
| tree | dfa7f6e7ab2220c179bfb3bb6faed842b592ca7d /indra/llui/llmodaldialog.cpp | |
| parent | e0c604c5f66e00aaa40ba6a418ff28a4a60af0b8 (diff) | |
BugSplat Crash #1409959 onTopLost
onTopLost crashed
1. It contradicts callstack, but clearPopups() definetely has an issue
due to not checking the pointer prior to calling onTopLost
2. According to callstack, crash happened around ~LLFolderViewFolder
and while it does call removePopup for itself, it isn't a popup, the
only one in the list would be the renamer, which calls back to parent,
so made sure to secure it.
3. mFlashTimer was never deleted
4. Some explicit cleanup for TopLost
Diffstat (limited to 'indra/llui/llmodaldialog.cpp')
| -rw-r--r-- | indra/llui/llmodaldialog.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/indra/llui/llmodaldialog.cpp b/indra/llui/llmodaldialog.cpp index 22d98469ec..c4dacd777e 100644 --- a/indra/llui/llmodaldialog.cpp +++ b/indra/llui/llmodaldialog.cpp @@ -67,6 +67,8 @@ LLModalDialog::~LLModalDialog() { LL_ERRS() << "Attempt to delete dialog while still in sModalStack!" << LL_ENDL; } + + LLUI::getInstance()->removePopup(this); } // virtual |