diff options
| author | RunitaiLinden <davep@lindenlab.com> | 2023-10-18 14:54:07 -0500 |
|---|---|---|
| committer | RunitaiLinden <davep@lindenlab.com> | 2023-10-18 14:54:07 -0500 |
| commit | 1fe76ad0b92329651b44c46667355a903854b783 (patch) | |
| tree | 404accd3a31d5fc4f601b43e981615778be6dfa8 /indra/newview/lltoolpie.cpp | |
| parent | dedda64d70f58dcd8dcab4ecf4f2ab0fd159adf0 (diff) | |
SL-20149 Potential fix for crash when unsitting on a reflection probe. Also fix for assertion in lltexturefetch.
Diffstat (limited to 'indra/newview/lltoolpie.cpp')
| -rw-r--r-- | indra/newview/lltoolpie.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/indra/newview/lltoolpie.cpp b/indra/newview/lltoolpie.cpp index 935d61f7ea..83a707472e 100644 --- a/indra/newview/lltoolpie.cpp +++ b/indra/newview/lltoolpie.cpp @@ -128,11 +128,9 @@ BOOL LLToolPie::handleMouseDown(S32 x, S32 y, MASK mask) // left mouse down always picks transparent (but see handleMouseUp). // Also see LLToolPie::handleHover() - priorities are a bit different there. // Todo: we need a more consistent set of rules to work with - if (transp_object == visible_object || !visible_object) + if (transp_object == visible_object || !visible_object || + !transp_object) // avoid potential for null dereference below, don't make assumptions about behavior of pickImmediate { - // Note: if transparent object is null, then visible object is also null - // since transparent pick includes non-tranpsarent one. - // !transparent_object check will be covered by transparent_object == visible_object. mPick = transparent_pick; } else |