diff --git a/autobuild.xml b/autobuild.xml index e2d5c23a76..7ad3b24dc6 100644 --- a/autobuild.xml +++ b/autobuild.xml @@ -1817,9 +1817,9 @@ archive hash - 26b0386266b7bdbf7907fa4ee8cb7b27 + 9f70048bed4c3fc487bee9e9f4276037 url - file:///opt/firestorm/llphysicsextensions_tpv-1.0.10206976532-darwin64-10206976532.tar.bz2 + file:///opt/firestorm/llphysicsextensions_tpv-1.0.11371371972-darwin64-11371371972.tar.bz2 name darwin64 @@ -1841,9 +1841,9 @@ archive hash - bd24b6c7c62922b06340ec73bd147483 + 31daa8f133168cc9ab500aa3a235b63c url - file:///c:/cygwin/opt/firestorm/llphysicsextensions_tpv-1.0.10206976532-windows64-10206976532.tar.bz2 + file:///c:/cygwin/opt/firestorm/llphysicsextensions_tpv-1.0.11371371972-windows64-11371371972.tar.bz2 name windows diff --git a/indra/llcommon/llerror.cpp b/indra/llcommon/llerror.cpp index 1767bb3a4a..5a3f2cbc5c 100644 --- a/indra/llcommon/llerror.cpp +++ b/indra/llcommon/llerror.cpp @@ -1659,19 +1659,3 @@ namespace LLError sLocalizedOutOfMemoryWarning = message; } } - -void crashdriver(void (*callback)(int*)) -{ - // The LLERROR_CRASH macro used to have inline code of the form: - //int* make_me_crash = NULL; - //*make_me_crash = 0; - - // But compilers are getting smart enough to recognize that, so we must - // assign to an address supplied by a separate source file. We could do - // the assignment here in crashdriver() -- but then BugSplat would group - // all LL_ERRS() crashes as the fault of this one function, instead of - // identifying the specific LL_ERRS() source line. So instead, do the - // assignment in a lambda in the caller's source. We just provide the - // nullptr target. - callback(nullptr); -} diff --git a/indra/llcommon/llerror.h b/indra/llcommon/llerror.h index e92a607076..db0d0f2f4c 100644 --- a/indra/llcommon/llerror.h +++ b/indra/llcommon/llerror.h @@ -420,9 +420,11 @@ typedef LLError::NoClassInfo _LL_CLASS_TO_LOG; #define LL_NEWLINE '\n' // Use this only in LL_ERRS or in a place that LL_ERRS may not be used -#define LLERROR_CRASH \ -{ \ - crashdriver([](int* ptr){ *ptr = 0; exit(*ptr); }); \ +#define LLERROR_CRASH \ +{ \ + int* make_me_crash = (int*)0xDEADBEEFDEADBEEFUL; \ + *make_me_crash = 0; \ + exit(*make_me_crash); \ } #define LL_ENDL \ @@ -524,7 +526,4 @@ LL_DEBUGS("SomeTag") performs the locking and map-searching ONCE, then caches the result in a static variable. */ -// used by LLERROR_CRASH -void crashdriver(void (*)(int*)); - #endif // LL_LLERROR_H diff --git a/indra/newview/app_settings/settings.xml b/indra/newview/app_settings/settings.xml index face692555..4721909fbb 100644 --- a/indra/newview/app_settings/settings.xml +++ b/indra/newview/app_settings/settings.xml @@ -1048,7 +1048,7 @@ Type Boolean Value - 0 + 1 FSSupportGroupChatPrefixTesting @@ -1059,7 +1059,7 @@ Type Boolean Value - 0 + 1 AutoCloseOOC diff --git a/indra/newview/llappviewer.cpp b/indra/newview/llappviewer.cpp index f9699faee9..7a39320fbe 100644 --- a/indra/newview/llappviewer.cpp +++ b/indra/newview/llappviewer.cpp @@ -1789,6 +1789,7 @@ bool LLAppViewer::doFrame() display(); + if (LLStartUp::getStartupState() > STATE_AGENT_WAIT) // FIRE-34590 - Bugsplat caused by updating maps before world is loaded. { LLPerfStats::RecordSceneTime T(LLPerfStats::StatType_t::RENDER_IDLE); LL_PROFILE_ZONE_NAMED_CATEGORY_APP("df Snapshot"); diff --git a/indra/newview/lldrawpoolwater.cpp b/indra/newview/lldrawpoolwater.cpp index 11e25e91b7..37de069fb6 100644 --- a/indra/newview/lldrawpoolwater.cpp +++ b/indra/newview/lldrawpoolwater.cpp @@ -152,6 +152,12 @@ void LLDrawPoolWater::renderPostDeferred(S32 pass) LLEnvironment& environment = LLEnvironment::instance(); LLSettingsWater::ptr_t pwater = environment.getCurrentWater(); LLSettingsSky::ptr_t psky = environment.getCurrentSky(); + // FIRE-34590 - Bugsplat Crash typically in startup state, due to null water. + if (!pwater || !psky) + { + LL_WARNS() << "LLDrawPoolWater::renderPostDeferred: water or sky settings not available" << LL_ENDL; + } + // LLVector3 light_dir = environment.getLightDirection(); bool sun_up = environment.getIsSunUp(); bool moon_up = environment.getIsMoonUp(); diff --git a/indra/newview/llvosurfacepatch.cpp b/indra/newview/llvosurfacepatch.cpp index 6312a27492..83e94c7e5b 100644 --- a/indra/newview/llvosurfacepatch.cpp +++ b/indra/newview/llvosurfacepatch.cpp @@ -1057,7 +1057,7 @@ void LLTerrainPartition::getGeometry(LLSpatialGroup* group) } const bool has_tangents = tangents_start.get() != nullptr; - if (has_tangents) + if (has_tangents && index_offset > 0) // FIRE-34672 OPENSIM bugsplat crash { LLStrider vertices = vertices_start; LLStrider normals = normals_start;