Coverity fix: sprintf is replaced by snprintf
authorkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 24 Feb 2011 15:37:59 +0000 (15:37 +0000)
committerkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 24 Feb 2011 15:37:59 +0000 (15:37 +0000)
PHOS/AliPHOS.cxx
PHOS/AliPHOSGridFile.cxx
PHOS/AliPHOSRawFitterv2.cxx
PHOS/AliPHOSRecPoint.cxx
PHOS/AliPHOSReconstructor.cxx
PHOS/AliPHOSTrackSegmentMakerv1.cxx
PHOS/AliPHOSv0.cxx

index 97b9104..7cded1e 100644 (file)
@@ -578,7 +578,7 @@ void AliPHOS::SetTreeAddress()
   // Links Hits in the Tree to Hits array
   TBranch *branch;
   char branchname[20];
-  sprintf(branchname,"%s",GetName());
+  snprintf(branchname,20,"%s",GetName());
   // Branch address for hit tree
     TTree *treeH = fLoader->TreeH();
   if (treeH) {
index 135b9cf..f1d1177 100644 (file)
@@ -150,7 +150,7 @@ void AliPHOSGridFile::ListEvents() const
   // list the available events for the current path and run selected
 
   char path[80] ;
-  sprintf(path, "%s/%s-%s/%s/%s/%s", fRoot.Data(), fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), fRun.Data()) ;
+  snprintf(path,80,"%s/%s-%s/%s/%s/%s", fRoot.Data(), fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), fRun.Data()) ;
   AliInfo(Form("Searching %s", path)) ;
 #if ROOT_VERSION_CODE < ROOT_VERSION(5,0,0)
   Grid_ResultHandle_t gr = fGrid->Find(path, "galice.root") ;
@@ -167,7 +167,7 @@ void AliPHOSGridFile::ListRuns() const
   // list the available runs for the current path selected
 
   char path[80] ;
-  sprintf(path, "%s/%s-%s/%s/%s", fRoot.Data(), fYear.Data(), fProd.Data(), fVers.Data(), fType.Data()) ;
+  snprintf(path,80,"%s/%s-%s/%s/%s", fRoot.Data(), fYear.Data(), fProd.Data(), fVers.Data(), fType.Data()) ;
   AliInfo(Form("Searching %s", path)) ;
 #if ROOT_VERSION_CODE < ROOT_VERSION(5,0,0)
   Grid_ResultHandle_t gr = fGrid->OpenDir(path) ;
@@ -184,7 +184,7 @@ Bool_t AliPHOSGridFile::SetYearProd(TString year, TString prod)
   // set the year and verifies if the directory exists
   Bool_t rv = kFALSE ;
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s", year.Data(), prod.Data()) ;
+  snprintf(tempo,80,"/%s-%s", year.Data(), prod.Data()) ;
 
   TString path(fRoot) ;
   path += tempo ;
@@ -209,7 +209,7 @@ Bool_t AliPHOSGridFile::SetVers(TString vers)
   // set the year and verifies if the directory exists
   Bool_t rv = kFALSE ;
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s/%s", fYear.Data(), fProd.Data(), vers.Data()) ;
+  snprintf(tempo,80,"/%s-%s/%s", fYear.Data(), fProd.Data(), vers.Data()) ;
   fVers = tempo ;
 
   TString path(fRoot) ;
@@ -234,7 +234,7 @@ Bool_t AliPHOSGridFile::SetType(TString type)
   // set the year and verifies if the directory exists
   Bool_t rv = kFALSE ;
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s/%s/%s", fYear.Data(), fProd.Data(), fVers.Data(), type.Data()) ;
+  snprintf(tempo,80,"/%s-%s/%s/%s", fYear.Data(), fProd.Data(), fVers.Data(), type.Data()) ;
 
   TString path(fRoot) ;
   path += tempo ;
@@ -258,7 +258,7 @@ Bool_t AliPHOSGridFile::SetPath(TString year, TString prod, TString vers, TStrin
   // set the year and verifies if the directory exists
   Bool_t rv = kFALSE ;
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s/%s/%s", year.Data(), prod.Data(), vers.Data(), type.Data()) ;
+  snprintf(tempo,80,"/%s-%s/%s/%s", year.Data(), prod.Data(), vers.Data(), type.Data()) ;
 
   TString path(fRoot) ;
   path += tempo ;
@@ -294,7 +294,7 @@ Bool_t AliPHOSGridFile::SetRun(Int_t run)
     srun.Prepend("0") ;
 
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s/%s/%s/%s", fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), srun.Data()) ;
+  snprintf(tempo,80,"/%s-%s/%s/%s/%s", fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), srun.Data()) ;
 
   TString path(fRoot) ;
   path += tempo ;
@@ -327,7 +327,7 @@ Bool_t AliPHOSGridFile::SetEvt(Int_t evt)
     sevt.Prepend("0") ;
 
   char tempo[80] ;
-  sprintf(tempo, "/%s-%s/%s/%s/%s/%s/", fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), fRun.Data(), sevt.Data()) ;
+  snprintf(tempo,80,"/%s-%s/%s/%s/%s/%s/", fYear.Data(), fProd.Data(), fVers.Data(), fType.Data(), fRun.Data(), sevt.Data()) ;
   TString path(fRoot) ;
   path += tempo ;
 #if ROOT_VERSION_CODE < ROOT_VERSION(5,0,0)
index 04a8b40..643ee74 100644 (file)
@@ -279,7 +279,7 @@ Bool_t AliPHOSRawFitterv2::Eval(const UShort_t *signal, Int_t sigStart, Int_t si
       spectrum_1->SetRightMargin(0.05);
 
       char title[155] ;
-      sprintf(title,"Sample, mod=%d, x=%d, z=%d, Quality=%5.1f",fModule,fCellX,fCellZ,fQuality) ;
+      snprintf(title,155,"Sample, mod=%d, x=%d, z=%d, Quality=%5.1f",fModule,fCellX,fCellZ,fQuality) ;
       h->SetTitle(title) ;
 //      h->Fit(fffit,"","",0.,51.) ;
       h->Draw() ;
index e84d789..c4778b6 100644 (file)
@@ -194,8 +194,8 @@ void AliPHOSRecPoint::ExecuteEvent(Int_t event, Int_t, Int_t)
       clustertext = new TPaveText(pos.X()-10,pos.Z()+10,pos.X()+50,pos.Z()+35,"") ;
       Text_t  line1[40] ;
       Text_t  line2[40] ;
-      sprintf(line1,"Energy=%1.2f GeV",GetEnergy()) ;
-      sprintf(line2,"%d Digits",GetDigitsMultiplicity()) ;
+      snprintf(line1,40,"Energy=%1.2f GeV",GetEnergy()) ;
+      snprintf(line2,40,"%d Digits",GetDigitsMultiplicity()) ;
       clustertext ->AddText(line1) ;
       clustertext ->AddText(line2) ;
       clustertext ->Draw("");
index ad1a31a..6a6e00c 100644 (file)
@@ -493,7 +493,7 @@ void AliPHOSReconstructor::FillMisalMatrixes(AliESDEvent* esd)const{
   char path[255] ;
   TGeoHMatrix * m ;
   for(Int_t mod=0; mod<5; mod++){
-    sprintf(path,"/ALIC_1/PHOS_%d",mod+1) ; //In Geometry modules numbered 1,2,.,5
+    snprintf(path,255,"/ALIC_1/PHOS_%d",mod+1) ; //In Geometry modules numbered 1,2,.,5
     if (gGeoManager->cd(path)){
       m = gGeoManager->GetCurrentMatrix() ;
       esd->SetPHOSMatrix(new TGeoHMatrix(*m),mod) ;
index c972aa2..bd4ada5 100644 (file)
@@ -415,7 +415,7 @@ void  AliPHOSTrackSegmentMakerv1::GetDistanceInPHOSPlane(AliPHOSEmcRecPoint * em
   
   emcClu->GetLocalPosition(vecEmc) ;
   
-  Double_t xCPV,zCPV ; //EMC-projected coordinates of CPV cluster 
+  Double_t xCPV=0,zCPV=0 ; //EMC-projected coordinates of CPV cluster 
   TVector3 cpvGlobal; // Global position of the CPV recpoint
   fGeom->GetGlobalPHOS((AliPHOSRecPoint*)cpvClu,cpvGlobal);
   Double_t vtxCPV[3]={cpvGlobal.X(),cpvGlobal.Y(),cpvGlobal.Z()} ;
index c96ff6d..b7b491d 100644 (file)
@@ -150,7 +150,7 @@ void AliPHOSv0::CreateGeometry()
   char im[5] ;
   Bool_t anyModuleCreated=0 ;
   for (Int_t iModule = 0; iModule < 5 ; iModule++ ) {
-    sprintf(im,"%d",iModule+1) ;
+    snprintf(im,5,"%d",iModule+1) ;
     if(strstr(GetTitle(),im)==0 && strcmp(GetTitle(),"IHEP")!=0 && strcmp(GetTitle(),"noCPV")!=0)
       continue ;
     anyModuleCreated=1 ;
@@ -736,7 +736,7 @@ void AliPHOSv0::AddAlignableVolumes() const
   
   char im[5] ;
   for(Int_t iModule=1; iModule<=nModules; iModule++){
-    sprintf(im,"%d",iModule) ;
+    snprintf(im,5,"%d",iModule) ;
     modUID = AliGeomManager::LayerToVolUID(idPHOS1,modnum++);
     if(strstr(GetTitle(),im)==0 && strcmp(GetTitle(),"IHEP")!=0 && strcmp(GetTitle(),"noCPV")!=0)
       continue ;
@@ -774,7 +774,7 @@ void AliPHOSv0::AddAlignableVolumes() const
   for(Int_t iModule=1; iModule<=nModules; iModule++){
     if(strstr(GetTitle(),"noCPV"))
       continue ;
-    sprintf(im,"%d",iModule) ;
+    snprintf(im,5,"%d",iModule) ;
     modUID = AliGeomManager::LayerToVolUID(idPHOS2,modnum++);
     if(strstr(GetTitle(),im)==0 && strcmp(GetTitle(),"IHEP")!=0)
       continue ;
@@ -848,7 +848,7 @@ void AliPHOSv0::AddAlignableVolumes() const
 
   for(Int_t module = 1; module <= nModules; ++module){
 
-    sprintf(im,"%d",module) ;
+    snprintf(im,5,"%d",module) ;
     if(strstr(GetTitle(),im)==0 && strcmp(GetTitle(),"IHEP")!=0 && strcmp(GetTitle(),"noCPV")!=0)
       continue ;
 
@@ -891,7 +891,7 @@ void AliPHOSv0::AddAlignableVolumes() const
          TGeoHMatrix *matTtoL = new TGeoHMatrix(matLtoT.Inverse());
  
          char phosPath[50] ;
-         sprintf(phosPath,"/ALIC_1/PHOS_%d",module) ;
+         snprintf(phosPath,50,"/ALIC_1/PHOS_%d",module) ;
          if (!gGeoManager->cd(phosPath)){
             AliFatal("Geo manager can not find path \n");
          }