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

#5102 Janitor : Guard null pointer access

Browse Source
This commit is contained in:
Magne Sjaastad 2019-11-27 10:17:31 +01:00
parent 5cfab3c497
commit bfdc84a744
9 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ApplicationCode/Commands/SummaryPlotCommands/RicPasteAsciiDataToSummaryPlotFeature.cpp
View File

@ -54,8 +54,7 @@ bool RicPasteAsciiDataToSummaryPlotFeature::isCommandEnabled()
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
RimSummaryPlot* summaryPlot = nullptr;
destinationObject->firstAncestorOrThisOfType( summaryPlot );
if ( !destinationObject ) return false;
RimSummaryPlotCollection* summaryPlotCollection = nullptr;
destinationObject->firstAncestorOrThisOfType( summaryPlotCollection );

1
ApplicationCode/Commands/SummaryPlotCommands/RicPasteEnsembleCurveSetFeature.cpp
View File

@ -67,6 +67,7 @@ RimEnsembleCurveSet*
bool RicPasteEnsembleCurveSetFeature::isCommandEnabled()
{
caf::PdmObject* destinationObject = dynamic_cast<caf::PdmObject*>( caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimSummaryPlot* plot = nullptr;
RimEnsembleCurveSetCollection* coll = nullptr;

3
ApplicationCode/Commands/SummaryPlotCommands/RicPasteSummaryCaseFeature.cpp
View File

@ -80,6 +80,7 @@ void RicPasteSummaryCaseFeature::onActionTriggered( bool isChecked )
{
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return;
std::vector<caf::PdmPointer<RimSummaryCase>> sourceObjects = RicPasteSummaryCaseFeature::summaryCases();
@ -162,4 +163,4 @@ void RicPasteSummaryCaseFeature::removeFromSourceCollection( RimSummaryCase* sum
sourceSummaryCaseMainCollection->removeCase( summaryCase );
sourceSummaryCaseMainCollection->updateConnectedEditors();
}
}
}

1
ApplicationCode/Commands/SummaryPlotCommands/RicPasteSummaryCrossPlotFeature.cpp
View File

@ -69,6 +69,7 @@ bool RicPasteSummaryCrossPlotFeature::isCommandEnabled()
{
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimSummaryCrossPlotCollection* plotColl = nullptr;
destinationObject->firstAncestorOrThisOfType( plotColl );

1
ApplicationCode/Commands/SummaryPlotCommands/RicPasteSummaryCurveFeature.cpp
View File

@ -74,6 +74,7 @@ RimSummaryCurve* RicPasteSummaryCurveFeature::copyCurveAndAddToPlot( RimSummaryC
bool RicPasteSummaryCurveFeature::isCommandEnabled()
{
caf::PdmObject* destinationObject = dynamic_cast<caf::PdmObject*>( caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimSummaryPlot* summaryPlot = nullptr;
destinationObject->firstAncestorOrThisOfType( summaryPlot );

2
ApplicationCode/Commands/SummaryPlotCommands/RicPasteSummaryPlotFeature.cpp
View File

@ -70,6 +70,8 @@ bool RicPasteSummaryPlotFeature::isCommandEnabled()
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimSummaryPlotCollection* plotColl = nullptr;
destinationObject->firstAncestorOrThisOfType( plotColl );
if ( !plotColl )

2
ApplicationCode/Commands/WellLogCommands/RicPasteWellLogCurveFeature.cpp
View File

@ -46,6 +46,7 @@ bool RicPasteWellLogCurveFeature::isCommandEnabled()
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimWellLogTrack* wellLogTrack = nullptr;
destinationObject->firstAncestorOrThisOfType( wellLogTrack );
@ -66,6 +67,7 @@ void RicPasteWellLogCurveFeature::onActionTriggered( bool isChecked )
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return;
RimWellLogTrack* wellLogTrack = nullptr;
destinationObject->firstAncestorOrThisOfType( wellLogTrack );

2
ApplicationCode/Commands/WellLogCommands/RicPasteWellLogPlotFeature.cpp
View File

@ -40,6 +40,7 @@ bool RicPasteWellLogPlotFeature::isCommandEnabled()
{
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimWellLogPlotCollection* wellLogPlotCollection = nullptr;
destinationObject->firstAncestorOrThisOfType( wellLogPlotCollection );
@ -58,6 +59,7 @@ void RicPasteWellLogPlotFeature::onActionTriggered( bool isChecked )
{
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return;
RimWellLogPlotCollection* wellLogPlotCollection = nullptr;
destinationObject->firstAncestorOrThisOfType( wellLogPlotCollection );

1
ApplicationCode/Commands/WellLogCommands/RicPasteWellLogTrackFeature.cpp
View File

@ -44,6 +44,7 @@ bool RicPasteWellLogTrackFeature::isCommandEnabled()
caf::PdmObjectHandle* destinationObject = dynamic_cast<caf::PdmObjectHandle*>(
caf::SelectionManager::instance()->selectedItem() );
if ( !destinationObject ) return false;
RimWellLogPlot* wellLogPlot = nullptr;
RimWellRftPlot* rftPlot = nullptr;