In AliMUONSlatGeometryBuilder:
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 29 Sep 2010 09:50:26 +0000 (09:50 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 29 Sep 2010 09:50:26 +0000 (09:50 +0000)
Added extra security on array index ;
this fixes OVERRUN_STATIC defect reported by Coverity.
(Javier)

MUON/AliMUONSlatGeometryBuilder.cxx

index aa0cdd6..0ed9e1b 100644 (file)
@@ -924,7 +924,7 @@ void AliMUONSlatGeometryBuilder::CreateGeometry()
        lCableY = ySlat3;
 
        // Cables going out from the start of slat
-       if(kNPCB3[i]>=4){ // Only if 4 or more pcb
+       if(kNPCB3[i]>=4 && i<kNslats3-2){ // Only if 4 or more pcb
          // First top cables
          cablepar[0] = supporthlength/2.;
          lCableX = 0.;
@@ -1537,7 +1537,7 @@ void AliMUONSlatGeometryBuilder::CreateGeometry()
       lCableY8 = ySlat42;
 
       // Cables going out from the start of slat
-      if(kNPCB4[i]>=4){ // Only if 4 or more pcb
+      if(kNPCB4[i]>=4 && i<kNslats4-2){ // Only if 4 or more pcb
        // First top cables
        cablepar[0] = (supporthlength-kXpos4[i])/2.;
        lCableX = kXpos4[i]/2.;