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/llfolderview.h | |
| 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/llfolderview.h')
| -rw-r--r-- | indra/llui/llfolderview.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/indra/llui/llfolderview.h b/indra/llui/llfolderview.h index 5f8a173889..f3c23ffc17 100644 --- a/indra/llui/llfolderview.h +++ b/indra/llui/llfolderview.h @@ -345,6 +345,8 @@ protected: LLUICtrl::CommitCallbackRegistry::ScopedRegistrar* mCallbackRegistrar; LLUICtrl::EnableCallbackRegistry::ScopedRegistrar* mEnableRegistrar; + boost::signals2::connection mRenamerTopLostSignalConnection; + bool mForceArrange; public: |