Fixing more coverity
authorfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 16 Jan 2012 15:33:30 +0000 (15:33 +0000)
committerfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 16 Jan 2012 15:33:30 +0000 (15:33 +0000)
STEER/CDB/AliSplineFit.cxx
STEER/STEERBase/AliTPCdEdxInfo.cxx
STRUCT/AliPIPEv4.cxx

index d03585f..e5c7545 100644 (file)
@@ -26,6 +26,7 @@
 
 
 #include "AliSplineFit.h"
+#include "AliLog.h"
 
 ClassImp(AliSplineFit)
 
@@ -267,11 +268,14 @@ TGraph * AliSplineFit::MakeGraph(Double_t xmin, Double_t xmax, Int_t npoints, In
   if (npoints<=0) {
     if (deriv<=0) 
       return new TGraph(fN,fX,fY0);
-    else 
-      if (deriv==1) 
-       return new TGraph(fN,fX,fY1);
-      else 
-       return new TGraph(fN-1,fX,fChi2I);
+    else if (deriv==1) 
+      return new TGraph(fN,fX,fY1);
+    else if(deriv>2)
+      return new TGraph(fN-1,fX,fChi2I);
+    else {
+      AliWarning("npoints == 0 et deriv == 2: unhandled condition");
+      return 0;
+    }
   }
   Double_t * x = new Double_t[npoints+1];
   Double_t * y = new Double_t[npoints+1];
index 8b22d6c..5e23394 100644 (file)
@@ -29,28 +29,6 @@ AliTPCdEdxInfo::AliTPCdEdxInfo():
 }
 
 
-AliTPCdEdxInfo::AliTPCdEdxInfo(const AliTPCdEdxInfo& source):
-  TObject(),
-  fTPCsignalRegion(),
-  fTPCsignalNRegion(),
-  fTPCsignalNRowRegion()
-{
-  //
-  // copy constructor
-  //
-  Double32_t signal[4]; Char_t ncl[3]; Char_t nrows[3];
-  source.GetTPCSignalRegionInfo(signal, ncl, nrows);
-  for (Int_t i=0;i<3; i++){
-    fTPCsignalRegion[i]=signal[i];
-    fTPCsignalNRegion[i]=ncl[i];
-    fTPCsignalNRowRegion[i]=nrows[i];
-  }
-  fTPCsignalRegion[3]=signal[3];
-}
-
-
-
 void  AliTPCdEdxInfo::GetTPCSignalRegionInfo(Double32_t signal[4], Char_t ncl[3], Char_t nrows[3]) const {
   //
   // Get the TPC dEdx variables per region
index c20f43d..71f0b23 100644 (file)
@@ -2925,45 +2925,45 @@ TGeoVolume* AliPIPEv4::MakeBellow(const char* ext, Int_t nc, Float_t rMin, Float
   const TGeoMedium* kMedSteel  =  gGeoManager->GetMedium("PIPE_INOX");   
 
   char name[64], nameA[64], nameB[64], bools[64];
-  sprintf(name, "%sBellowUS", ext);
+  snprintf(name, 64, "%sBellowUS", ext);
   TGeoVolume* voBellow = new TGeoVolume(name, new TGeoTube(rMin, rMax, dU/2.), kMedVac);
   //      
   //  Upper part of the undulation
   //
   TGeoTorus* shPlieTorusU  =  new TGeoTorus(rMax - rPlie, rPlie - dPlie, rPlie);
-  sprintf(nameA, "%sTorusU", ext);
+  snprintf(nameA, 64, "%sTorusU", ext);
   shPlieTorusU->SetName(nameA);
   TGeoTube*  shPlieTubeU   =  new TGeoTube (rMax - rPlie, rMax, rPlie);
-  sprintf(nameB, "%sTubeU", ext);
+  snprintf(nameB, 64, "%sTubeU", ext);
   shPlieTubeU->SetName(nameB);
-  sprintf(name, "%sUpperPlie", ext);
-  sprintf(bools, "%s*%s", nameA, nameB);
+  snprintf(name, 64, "%sUpperPlie", ext);
+  snprintf(bools, 64, "%s*%s", nameA, nameB);
   TGeoCompositeShape*  shUpperPlie = new TGeoCompositeShape(name, bools);
     
   TGeoVolume* voWiggleU = new TGeoVolume(name, shUpperPlie, kMedSteel);
   //
   // Lower part of the undulation
   TGeoTorus* shPlieTorusL =  new TGeoTorus(rMin + rPlie, rPlie - dPlie, rPlie);
-  sprintf(nameA, "%sTorusL", ext);
+  snprintf(nameA, 64, "%sTorusL", ext);
   shPlieTorusL->SetName(nameA);
   TGeoTube*  shPlieTubeL  =  new TGeoTube (rMin, rMin + rPlie, rPlie);
-  sprintf(nameB, "%sTubeL", ext);
+  snprintf(nameB, 64, "%sTubeL", ext);
   shPlieTubeL->SetName(nameB);
-  sprintf(name, "%sLowerPlie", ext);
-  sprintf(bools, "%s*%s", nameA, nameB);
+  snprintf(name, 64, "%sLowerPlie", ext);
+  snprintf(bools, 64, "%s*%s", nameA, nameB);
   TGeoCompositeShape*  shLowerPlie = new TGeoCompositeShape(name, bools);
     
   TGeoVolume* voWiggleL = new TGeoVolume(name, shLowerPlie, kMedSteel); 
     
   //
   // Connection between upper and lower part of undulation
-  sprintf(name, "%sPlieConn1", ext);
+  snprintf(name, 64, "%sPlieConn1", ext);
   TGeoVolume* voWiggleC1 = new TGeoVolume(name, new TGeoTube(rMin + rPlie, rMax - rPlie, dPlie/2.), kMedSteel);
   //
   // One wiggle
   Float_t dz = rPlie -  dPlie / 2.;
   Float_t z0 = -  dPlie / 2.;
-  sprintf(name, "%sWiggle", ext);
+  snprintf(name, 64, "%sWiggle", ext);
   TGeoVolumeAssembly* asWiggle = new TGeoVolumeAssembly(name);
   asWiggle->AddNode(voWiggleC1,  1 , new TGeoTranslation(0., 0., z0));
   z0 += dz;