Fixing for coverity report
authorcoppedis <coppedis@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 24 Jan 2011 11:50:32 +0000 (11:50 +0000)
committercoppedis <coppedis@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 24 Jan 2011 11:50:32 +0000 (11:50 +0000)
ZDC/AliZDC.cxx
ZDC/AliZDCv3.cxx

index f891fad..599fb03 100644 (file)
@@ -539,7 +539,10 @@ void AliZDC::Digits2Raw()
     // digit.GetSector(0),digit.GetSector(1),lADCDataGEO,lADCDataChannel);
      
     if(lADCDataGEO==0){ 
-      if(indADC0>knADCData1) AliError(" Problem with digit index 4 ADC0\n");
+      if(indADC0>knADCData1){
+        AliError(" Problem with digit index 4 ADC0\n");
+       return;
+      }
       Int_t indLG = indADC0+knADCData1;
       // High gain ADC ch.      
       if(digit.GetADCValue(0) > 2047) lADCDataOvFlwHG = 1; 
@@ -548,7 +551,7 @@ void AliZDC::Digits2Raw()
                    lADCDataOvFlwHG << 12 | (lADCDataValue1[indADC0] & 0xfff); 
       // Low gain ADC ch.
       if(digit.GetADCValue(1) > 2047) lADCDataOvFlwLG = 1; 
-      lADCDataValue1[indLG] = digit.GetADCValue(1); 
+      lADCDataValue1[indLG-1] = digit.GetADCValue(1); 
       lADCData1[indLG] = lADCDataGEO << 27 |  lADCDataChannel << 17 | 0x1 << 16 |
                    lADCDataOvFlwLG << 12 | (lADCDataValue1[indLG] & 0xfff);  
                    
@@ -559,7 +562,10 @@ void AliZDC::Digits2Raw()
       //  lADCDataGEO,iDigit,lADCDataValue1[indLG],indLG,lADCDataValue1[indLG]);
     }
     else if(lADCDataGEO==1){ 
-      if(indADC1>knADCData2) AliError(" Problem with digit index 4 ADC1\n");
+      if(indADC1>knADCData2){
+         AliError(" Problem with digit index 4 ADC1\n");
+        return;
+      }
       Int_t indLG = indADC1+knADCData2;
       // High gain ADC ch.      
       if(digit.GetADCValue(0) > 2047) lADCDataOvFlwHG = 1; 
@@ -579,7 +585,10 @@ void AliZDC::Digits2Raw()
       //  lADCDataGEO,indHG,lADCDataValue2[indHG],indLG,lADCDataValue2[indLG]);
     }
     else if(lADCDataGEO==2){ 
-      if(indADC2>knADCData2) AliError(" Problem with digit index 4 ADC2\n");
+      if(indADC2>knADCData2){
+        AliError(" Problem with digit index 4 ADC2\n");
+       return;
+      }
       Int_t indLG = indADC2+knADCData3;
       // High gain ADC ch.      
       if(digit.GetADCValue(0) > 2047) lADCDataOvFlwHG = 1; 
@@ -599,7 +608,10 @@ void AliZDC::Digits2Raw()
       //  lADCDataGEO,indHG,lADCDataValue3[indHG],indLG,lADCDataValue3[indLG]);
     }
     else if(lADCDataGEO==3){ 
-      if(indADC3>knADCData3) AliError(" Problem with digit index 4 ADC2\n");
+      if(indADC3>knADCData3){
+         AliError(" Problem with digit index 4 ADC2\n");
+        return;
+      }
       Int_t indLG = indADC3+knADCData4;
       // High gain ADC ch.      
       if(digit.GetADCValue(0) > 2047) lADCDataOvFlwHG = 1; 
index 40c8ef3..e7fa9ee 100644 (file)
@@ -2200,28 +2200,28 @@ void AliZDCv3::InitTables()
 
   Int_t k, j;
 
-  char *lightfName1,*lightfName2,*lightfName3,*lightfName4,
-       *lightfName5,*lightfName6,*lightfName7,*lightfName8;
-  FILE *fp1, *fp2, *fp3, *fp4, *fp5, *fp6, *fp7, *fp8;
-
   //  --- Reading light tables for ZN 
-  lightfName1 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362207s");
-  if((fp1 = fopen(lightfName1,"r")) == NULL){
+  char *lightfName1 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362207s");
+  FILE *fp1 = fopen(lightfName1,"r");
+  if(fp1 == NULL){
      printf("Cannot open file fp1 \n");
      return;
   }
-  lightfName2 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362208s");
-  if((fp2 = fopen(lightfName2,"r")) == NULL){
+  char *lightfName2 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362208s");
+  FILE *fp2 = fopen(lightfName2,"r");
+  if(fp2 == NULL){
      printf("Cannot open file fp2 \n");
      return;
   }  
-  lightfName3 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362209s");
-  if((fp3 = fopen(lightfName3,"r")) == NULL){
+  char *lightfName3 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362209s");
+  FILE *fp3 = fopen(lightfName3,"r");
+  if(fp3 == NULL){
      printf("Cannot open file fp3 \n");
      return;
   }
-  lightfName4 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362210s");
-  if((fp4 = fopen(lightfName4,"r")) == NULL){
+  char *lightfName4 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620362210s");
+  FILE *fp4 = fopen(lightfName4,"r");
+  if(fp4 == NULL){
      printf("Cannot open file fp4 \n");
      return;
   }
@@ -2245,23 +2245,27 @@ void AliZDCv3::InitTables()
   fclose(fp4);
   
   //  --- Reading light tables for ZP and ZEM
-  lightfName5 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552207s");
-  if((fp5 = fopen(lightfName5,"r")) == NULL){
+  char *lightfName5 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552207s");
+  FILE *fp5 = fopen(lightfName5,"r");
+  if(fp5 == NULL){
      printf("Cannot open file fp5 \n");
      return;
   }
-  lightfName6 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552208s");
-  if((fp6 = fopen(lightfName6,"r")) == NULL){
+  char *lightfName6 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552208s");
+  FILE *fp6 = fopen(lightfName6,"r");
+  if(fp6 == NULL){
      printf("Cannot open file fp6 \n");
      return;
   }
-  lightfName7 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552209s");
-  if((fp7 = fopen(lightfName7,"r")) == NULL){
+  char *lightfName7 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552209s");
+  FILE *fp7 = fopen(lightfName7,"r");
+  if(fp7 == NULL){
      printf("Cannot open file fp7 \n");
      return;
   }
-  lightfName8 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552210s");
-  if((fp8 = fopen(lightfName8,"r")) == NULL){
+  char *lightfName8 = gSystem->ExpandPathName("$ALICE_ROOT/ZDC/light22620552210s");
+  FILE *fp8 = fopen(lightfName8,"r");
+  if(fp8 == NULL){
      printf("Cannot open file fp8 \n");
      return;
   }
@@ -2290,7 +2294,7 @@ void AliZDCv3::StepManager()
   // Routine called at every step in the Zero Degree Calorimeters
   //
   Int_t   j, vol[2]={0,0}, ibeta=0, ialfa=0, ibe=0, nphe=0;
-  Float_t hits[13], x[3], xdet[3], um[3], ud[3];
+  Float_t hits[13], x[3], xdet[3]={999.,999.,999.}, um[3], ud[3];
   Float_t destep=0., be=0., out=0.;
   Double_t s[3], p[4];
   const char *knamed;