mirror of
https://github.com/OPM/ResInsight.git
synced 2026-10-25 06:38:13 -05:00
Magne Sjaastad
b22a2e3106
RimViewController::masterView() dereferenced ownerViewLinker() without a null check, causing a crash when the controller has been detached from its RimViewLinker parent (e.g. mid-destruction, after the owning child-array field has already erased it but before delete runs). The same unguarded ownerViewLinker() dereference existed in several other methods: isActive(), isCameraLinked(), isTimeStepLinked(), isResultColorControlled(), isLegendDefinitionsControlled(), isPropertyFilterOveridden(), updateOverrides(), updateDuplicatedPropertyFilters(), updateCameraLink(), updateTimeStepLink(), updateResultColorsControl(), updateLegendDefinitions() and applyCellFilterCollectionByUserChoice(). All now store ownerViewLinker() in a local and null-check it before use, matching the pattern already used in isCellFiltersControlled().