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/newview/llsplitbutton.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/newview/llsplitbutton.h')
| -rw-r--r-- | indra/newview/llsplitbutton.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/indra/newview/llsplitbutton.h b/indra/newview/llsplitbutton.h index 4f20c8b379..dbb18c0c62 100644 --- a/indra/newview/llsplitbutton.h +++ b/indra/newview/llsplitbutton.h @@ -67,7 +67,7 @@ public: }; - virtual ~LLSplitButton() {}; + virtual ~LLSplitButton(); //Overridden virtual void onFocusLost(); @@ -99,6 +99,8 @@ protected: LLButton* mShownItem; EArrowPosition mArrowPosition; + boost::signals2::connection mTopLostSignalConnection; + commit_callback_t mSelectionCallback; }; |