OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2026-08-27 05:37:21 -05:00
Code Issues Packages Projects Releases Wiki Activity

#14006 CorrelationPlot: Guard against empty time steps in fieldChangedByUi

Browse Source 
Changing the time step filter combo box crashed when the plot had no
available time steps: filteredTimeStepIndices returned an empty vector
and .back() invoked undefined behaviour. Skip the m_timeStep assignment
when either allDateTimes or filteredTimeStepIndices is empty.

Fixes #14006.
This commit is contained in:
Kristian Bendiksen
2026-05-20 18:04:01 +02:00
committed by Magne Sjaastad
parent 37c6892102
commit cc574e7a7b
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ApplicationLibCode/ProjectDataModel/CorrelationPlots/RimAbstractCorrelationPlot.cpp
+9 -3
View File
@@ -158,10 +158,16 @@ void RimAbstractCorrelationPlot::fieldChangedByUi( const caf::PdmFieldHandle* ch
allDateTimes.push_back( dateTime );
}
std::vector<int> filteredTimeStepIndices =
RimTimeStepFilter::filteredTimeStepIndices( allDateTimes, 0, (int)allDateTimes.size() - 1, m_timeStepFilter(), 1 );
if ( !allDateTimes.empty() )
{
std::vector<int> filteredTimeStepIndices =
RimTimeStepFilter::filteredTimeStepIndices( allDateTimes, 0, (int)allDateTimes.size() - 1, m_timeStepFilter(), 1 );
m_timeStep = allDateTimes[filteredTimeStepIndices.back()];
if ( !filteredTimeStepIndices.empty() )
{
m_timeStep = allDateTimes[filteredTimeStepIndices.back()];
}
}
updateConnectedEditors();
}