OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2025-02-25 18:55:39 -06:00
Code Issues Packages Projects Releases Wiki Activity

#8041 Histogram Calculations : Guard access to missing or empty result data

Browse Source 
Crash seen when computing histogram data based on a difference case with no data.
This commit is contained in:
Magne Sjaastad 2021-09-23 11:18:57 +02:00
parent 515168600f
commit cb6180b118
2 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ApplicationLibCode/ReservoirDataModel/RigEclipseNativeVisibleCellsStatCalc.cpp
View File

@ -115,6 +115,9 @@ void RigEclipseNativeVisibleCellsStatCalc::mobileVolumeWeightedMean( size_t time
m_caseData->ensureKnownResultLoaded( mobPorvAddress );
if ( !m_caseData->hasResultEntry( m_resultAddress ) ) return;
if ( m_caseData->timeStepCount( m_resultAddress ) == 0 ) return;
const std::vector<double>& weights = m_caseData->cellScalarResults( mobPorvAddress, 0 );
const std::vector<double>& values = m_caseData->cellScalarResults( m_resultAddress, timeStepIndex );

3
ApplicationLibCode/ReservoirDataModel/RigEclipseNativeVisibleCellsStatCalc.h
View File

@ -52,6 +52,9 @@ private:
template <typename StatisticsAccumulator>
void traverseCells( StatisticsAccumulator& accumulator, size_t timeStepIndex )
{
if ( !m_caseData->hasResultEntry( m_resultAddress ) ) return;
if ( m_caseData->timeStepCount( m_resultAddress ) == 0 ) return;
size_t clampedTimeStepIndex = std::min( timeStepIndex, m_caseData->timeStepCount( m_resultAddress ) - 1 );
const std::vector<double>& values = m_caseData->cellScalarResults( m_resultAddress, clampedTimeStepIndex );