coverity fixes + bug fixes
authoramastros <amastros@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 28 Mar 2011 18:57:25 +0000 (18:57 +0000)
committeramastros <amastros@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 28 Mar 2011 18:57:25 +0000 (18:57 +0000)
ITS/UPGRADE/AliITSUpgradeClusterFinder.cxx
ITS/UPGRADE/AliITSUpgradeClusterList.cxx
ITS/UPGRADE/AliITSUpgradeClusterList.h

index e79dc80..79ed05c 100644 (file)
@@ -285,7 +285,7 @@ Int_t AliITSUpgradeClusterFinder::DoModuleClustering(Int_t module, UShort_t char
       if(size>1) AliDebug(2,Form("DoModuleClustering, size %i , labels :  %i  %i  %i \n",size,fLabels[0],fLabels[1],fLabels[2]));
       fClusterList[module]->Insert((Float_t)fColSum/size, (Float_t)fRowSum/size, size, GetClusterWidthZ(), GetClusterWidthPhi(), GetClusterType(size), fCharge,fLabels);
       fCharge=0;
-      for(Int_t i=0; i<10; i++) fLabels[i]=-5;
+      for(Int_t i=0; i<kMAXCLUSTERTYPESIDEZ*kMAXCLUSTERTYPESIDEY; i++) fLabels[i]=-5;
     }
     if (fNhitsLeft==0) break;
   }
@@ -784,7 +784,7 @@ void AliITSUpgradeClusterFinder::SetLabels(Int_t label[3]){
   for(Int_t k=0; k<3; k++){
     if(label[k]<0) continue;
     for(Int_t i=0; i<kMAXCLUSTERTYPESIDEZ*kMAXCLUSTERTYPESIDEY; i++){
-      if(position>kMAXCLUSTERTYPESIDEZ*kMAXCLUSTERTYPESIDEY) continue;
+      if(position>=kMAXCLUSTERTYPESIDEZ*kMAXCLUSTERTYPESIDEY) continue;
       // if there is no label assigned and it is not present in previous labels
       if(fLabels[position]<0) {
         if(!isAssigned[k]) {
index 8f7e623..c681396 100644 (file)
@@ -105,11 +105,11 @@ AliITSUpgradeClusterListNode* AliITSUpgradeClusterList::CloneNode(AliITSUpgradeC
   else return new AliITSUpgradeClusterListNode(node->Col(),node->Row(),node->Size(),node->WidthZ(),node->WidthPhi(),node->Type(),node->Charge(),CloneNode(node->Next()));
 }
 //______________________________________________________________________________
-Bool_t AliITSUpgradeClusterList::Insert(Float_t col, Float_t row, UShort_t size, UShort_t widthZ, UShort_t widthPhi, UShort_t type, UShort_t charge, Int_t digLabels[10]) {
+Bool_t AliITSUpgradeClusterList::Insert(Float_t col, Float_t row, UShort_t size, UShort_t widthZ, UShort_t widthPhi, UShort_t type, UShort_t charge, Int_t digLabels[12]) {
   // insert a new node into the list (returns true if the node was not present before)
   fNrEntries++;
   AliITSUpgradeClusterListNode* node = new AliITSUpgradeClusterListNode(col,row,size,widthZ,widthPhi,type,charge,NULL);
-  for(Int_t i=0; i< size; i++) node->AddDigitLabel(digLabels[i]); // adding digit label to the cluster
+  for(Int_t i=0; i< 12; i++) node->AddDigitLabel(digLabels[i]); // adding digit label to the cluster
   if (fFirst==NULL) {
     fFirst = node;
   }
index d3c1896..d21c844 100644 (file)
@@ -27,7 +27,7 @@ class AliITSUpgradeClusterList {
 
   void                   Clear();
   AliITSUpgradeClusterList*     Clone() const;
- Bool_t                 Insert(Float_t col, Float_t row, UShort_t size, UShort_t widthZ, UShort_t widthPhi, UShort_t type, UShort_t charge, Int_t labels[10]);
+ Bool_t                 Insert(Float_t col, Float_t row, UShort_t size, UShort_t widthZ, UShort_t widthPhi, UShort_t type, UShort_t charge, Int_t labels[12]);
 
   UInt_t   GetNrEntries() const {return fNrEntries;}
   Float_t  GetColIndex(UInt_t index);