From ca9b4618134f60704e10094d0bfbd103b6d93fcf Mon Sep 17 00:00:00 2001 From: sgorbuno Date: Sat, 12 Mar 2011 10:13:42 +0000 Subject: [PATCH] coverity warning 10074 fixed --- HLT/ITS/clusterfinders/AliHLTITSClusterFinderSSD.cxx | 4 ++++ HLT/ITS/tracking/AliHLTITSTrackerComponent.cxx | 9 ++++----- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/HLT/ITS/clusterfinders/AliHLTITSClusterFinderSSD.cxx b/HLT/ITS/clusterfinders/AliHLTITSClusterFinderSSD.cxx index 0e587fccb2b..05179d0cc09 100644 --- a/HLT/ITS/clusterfinders/AliHLTITSClusterFinderSSD.cxx +++ b/HLT/ITS/clusterfinders/AliHLTITSClusterFinderSSD.cxx @@ -374,6 +374,10 @@ FindClustersSSD(Ali1Dcluster* neg, Int_t nn, const TGeoHMatrix *mT2L=AliITSgeomTGeo::GetTracking2LocalMatrix(fModule); AliITSsegmentationSSD *seg = dynamic_cast(fDetTypeRec->GetSegmentationModel(2)); + if(!mT2L || !seg){ + AliError(Form("HLT ClustersFinderSSD: null pointer from TGeo")); + return; + } if (fModule>fLastSSD1) seg->SetLayer(6); else diff --git a/HLT/ITS/tracking/AliHLTITSTrackerComponent.cxx b/HLT/ITS/tracking/AliHLTITSTrackerComponent.cxx index bab14d6ef3e..6e8c6cb6ba5 100644 --- a/HLT/ITS/tracking/AliHLTITSTrackerComponent.cxx +++ b/HLT/ITS/tracking/AliHLTITSTrackerComponent.cxx @@ -249,25 +249,24 @@ int AliHLTITSTrackerComponent::Configure( const char* cdbEntry, const char* chai //* read magnetic field - int iResult2 = 0; //ReadCDBEntry( kAliHLTCDBSolenoidBz, chainId ); fSolenoidBz = GetBz(); //* read the actual CDB entry if required - int iResult3 = ( cdbEntry ) ? ReadCDBEntry( cdbEntry, chainId ) : 0; + int iResult2 = ( cdbEntry ) ? ReadCDBEntry( cdbEntry, chainId ) : 0; //* read extra parameters from input (if they are) - int iResult4 = 0; + int iResult3 = 0; if ( commandLine && commandLine[0] != '\0' ) { HLTInfo( "received configuration string from HLT framework: \"%s\"", commandLine ); - iResult4 = ReadConfigurationString( commandLine ); + iResult3 = ReadConfigurationString( commandLine ); } // Initialise the tracker here - return iResult1 ? iResult1 : ( iResult2 ? iResult2 : ( iResult3 ? iResult3 : iResult4 ) ); + return iResult1 ? iResult1 : ( iResult2 ? iResult2 : iResult3 ); } -- 2.43.0