some coverity warnings fixed
authoralla <alla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 3 Mar 2011 15:20:26 +0000 (15:20 +0000)
committeralla <alla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 3 Mar 2011 15:20:26 +0000 (15:20 +0000)
T0/AliT0.cxx
T0/AliT0Align.cxx
T0/AliT0QADataMakerRec.cxx
T0/AliT0v1.cxx

index 0b88c23..0d49cb5 100644 (file)
@@ -180,8 +180,8 @@ void AliT0::MakeBranch(Option_t* option)
   //    R         Make a branch of  AliT0RecPoints
   //
   char branchname[20];
-  sprintf(branchname,"%s",GetName());
-
+  // sprintf(branchname,"%s",GetName());
+  strncpy(branchname, GetName(), 20);
   const char *cH = strstr(option,"H");
   const char *cD = strstr(option,"D");
   const char *cR = strstr(option,"R");
@@ -280,7 +280,8 @@ void AliT0::MakeBranchInTreeD(TTree *treeD, const char *file)
     //
     const Int_t kBufferSize = 4000;
     char branchname[20];
-    sprintf(branchname,"%s",GetName());
+   strncpy(branchname, GetName(), 20);
+   //   sprintf(branchname,"%s",GetName());
     if(treeD)
      {
        MakeBranchInTree(treeD,  branchname,&fDigits, kBufferSize, file);
index d1f5021..f7a9067 100644 (file)
@@ -28,6 +28,7 @@ Preliminary test version (T.Malkiewicz)
 #include "TFile.h"
 #include "TMath.h"
 #include "TSystem.h"
+#include "TString.h"
 #include "AliSurveyObj.h"
 #include "AliAlignObjParams.h"
 #include "AliCDBStorage.h"
@@ -116,7 +117,7 @@ AliT0Align::AliT0Align(const AliT0Align &align) :
   //
   //  copy constructor - dummy
   //
-  // fDebug = align.fDebug;
+  fDebug = align.fDebug;
 }
 //__________________________________________________________________________
 AliT0Align & AliT0Align::operator =(const AliT0Align & align)
@@ -124,7 +125,7 @@ AliT0Align & AliT0Align::operator =(const AliT0Align & align)
   //
   // assignment operator - dummy
   //
-  // fDebug=align.fDebug;
+  fDebug=align.fDebug;
   return (*this);
 }
 
@@ -292,9 +293,10 @@ void AliT0Align::StoreAlignObj()
   if( TString(gSystem->Getenv("TOCDB")) != TString("kTRUE") ){
     // save on file
     const char* filename = "T0SurveyMisalignment.root";
-    Char_t fullname[80];
-    sprintf(fullname,"%s/T0/Align/Data/%s",gSystem->Getenv("ALICE_ROOT"),filename);
-    TFile *f = new TFile(fullname,"RECREATE");
+    //  Char_t fullname[80];
+    //  sprintf(fullname,"%s/T0/Align/Data/%s",gSystem->Getenv("ALICE_ROOT"),filename);
+    TString fullname = Form("%s/T0/Align/Data/%s",gSystem->Getenv("ALICE_ROOT"), filename);
+    TFile *f = new TFile(fullname.Data(),"RECREATE");
     if(!f){
       AliError("cannot open file for output\n");
       return;
index aa2baa7..1d363af 100644 (file)
@@ -66,9 +66,13 @@ AliQADataMakerRec(AliQAv1::GetDetName(AliQAv1::kT0),
       feffA[i]=0;
       feffqtc[i]=0;
       feffPhysC[i]=0;
+      feffPhysA[i]=0;
+      feffqtcPhys[i]=0;
+
    }
 }
 
+
 //____________________________________________________________________________ 
 AliT0QADataMakerRec::AliT0QADataMakerRec(const AliT0QADataMakerRec& qadm) :
   AliQADataMakerRec(),
index 376ca8b..5181216 100644 (file)
@@ -41,6 +41,7 @@
 #include <TLorentzVector.h>
 #include <TMath.h>
 #include <TVirtualMC.h>
+#include <TString.h>
 
 #include "AliLog.h"
 #include "AliMagF.h"
@@ -261,12 +262,14 @@ void AliT0v1::CreateGeometry()
     new TGeoTube("0HOLE", 0, 1.51, 6.5);
     TGeoTranslation *tr [12];
     Double_t angle  = 2 * TMath::Pi() / 12;
-    Char_t nameTr[40];
+    // Char_t nameTr[40];
+    TString nameTr;
     for (Int_t itr=0; itr<12; itr++) {
-      sprintf (nameTr,"0TR%i",itr+1);
+      //      sprintf (nameTr,"0TR%i",itr+1);
+      nameTr = Form("0TR%i",itr+1);
       x = 6.5 * TMath::Sin(itr * angle);
       y = 6.5 * TMath::Cos(itr * angle);
-      tr[itr] = new TGeoTranslation(nameTr,x,y,6.5);
+      tr[itr] = new TGeoTranslation(nameTr.Data(),x,y,6.5);
       tr[itr]->RegisterYourself();
       stlin->AddNode(ins,itr+13,tr[itr]);
     }