Coverity
authorfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 4 Feb 2011 11:24:48 +0000 (11:24 +0000)
committerfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 4 Feb 2011 11:24:48 +0000 (11:24 +0000)
13 files changed:
EVE/EveBase/AliEveCascadeEditor.cxx
EVE/EveBase/AliEveCascadeListEditor.cxx
EVE/EveBase/AliEveCosmicRayFitter.cxx
EVE/EveBase/AliEveCosmicRayFitterEditor.cxx
EVE/EveBase/AliEveEventManager.cxx
EVE/EveBase/AliEveEventManagerEditor.cxx
EVE/EveBase/AliEveHFEditor.cxx
EVE/EveBase/AliEveHFListEditor.cxx
EVE/EveBase/AliEveJetPlaneEditor.cxx
EVE/EveBase/AliEveKineTools.cxx
EVE/EveBase/AliEveKinkEditor.cxx
EVE/EveDet/AliEveITSScaledModule.cxx
STEER/AliRun.cxx

index 355ddb0..19d9fe8 100644 (file)
@@ -98,7 +98,7 @@ void AliEveCascadeEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveCascade*>(obj);
+  fM = static_cast<AliEveCascade*>(obj);
 
   // Set values of widgets
   fInfoLabelRadius->SetText(Form("Radius = %f cm",     fM->GetRadius() ));
index 4104518..98ebeab 100644 (file)
@@ -114,7 +114,7 @@ void AliEveCascadeListEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveCascadeList*>(obj);
+  fM = static_cast<AliEveCascadeList*>(obj);
 
   // Set values of widgets
   // fXYZZ->SetValue(fM->GetXYZZ());
index 78942fd..bc328df 100644 (file)
@@ -150,6 +150,7 @@ void AliEveCosmicRayFitter::AddFitPoint(Int_t n)
 
   Float_t x, y, z;
   TEvePointSet* ps = dynamic_cast<TEvePointSet*>((TQObject*) gTQSender);
+  if(!ps) AliFatal("ps == 0, dynamic cast failed\n");
 
   std::map<Point_t, Int_t>::iterator g = fSPMap.find(Point_t(ps, n));
   if(g != fSPMap.end())
@@ -363,8 +364,11 @@ void AliEveCosmicRayFitter::DrawDebugGraph()
   // draw graphs
 
   TCanvas * canvas = 0;
-  if (gPad) gPad->Clear();
-  else if (gPad==0 || gPad->GetCanvas()->IsEditable() == kFALSE) {
+  if (gPad) {
+         gPad->Clear();
+         canvas=gPad->GetCanvas();
+  }
+  if (gPad==0 || gPad->GetCanvas()->IsEditable() == kFALSE) {
     canvas = new TCanvas("canvas", "CosmicRayFitter", 800, 400);
     canvas->Clear();
   }
index 770d2d0..961fb9e 100644 (file)
@@ -65,7 +65,7 @@ void AliEveCosmicRayFitterEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveCosmicRayFitter*>(obj);
+  fM = static_cast<AliEveCosmicRayFitter*>(obj);
 
   if(fM->GetConnected())
   {
index 4d38ed3..f5ac2c0 100644 (file)
@@ -1567,6 +1567,7 @@ Bool_t AliEveEventManager::InsertGlobal(const TString& tag, TEveElement* model,
          model->SetRnrChildren(kFALSE);
 
          TEveElement* old_model = dynamic_cast<TEveElement*>(pair->Value());
+                if(!old_model) AliFatal("old_model == 0, dynamic cast failed\n");
          while (old_model->HasChildren())
          {
             TEveElement *el = old_model->FirstChild();
index f45b0f1..07f7e18 100644 (file)
@@ -71,7 +71,7 @@ void AliEveEventManagerEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveEventManager*>(obj);
+  fM = static_cast<AliEveEventManager*>(obj);
 
   fEventInfo->LoadBuffer(fM->GetEventInfoVertical());
 }
index 27073f8..1e2a9e0 100644 (file)
@@ -73,7 +73,7 @@ void AliEveHFEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveHF*>(obj);
+  fM = static_cast<AliEveHF*>(obj);
 
   // Set values of widgets
   fInfoLabel0->SetText(Form("CosPointingAngle = %f",  fM->GetCosPointingAngle()));
index e9e3200..dc5c192 100644 (file)
@@ -90,7 +90,7 @@ void AliEveHFListEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveHFList*>(obj);
+  fM = static_cast<AliEveHFList*>(obj);
 
   // Set values of widgets
   // fXYZZ->SetValue(fM->GetXYZZ());
index baf4360..b44e235 100644 (file)
@@ -100,7 +100,7 @@ void AliEveJetPlaneEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveJetPlane*>(obj);
+  fM = static_cast<AliEveJetPlane*>(obj);
 
   // Set values of widgets
   // fXYZZ->SetValue(fM->GetXYZZ());
index d763905..8e3881c 100644 (file)
@@ -43,7 +43,7 @@ namespace {
   {
     TEveElement::List_i i = cont->BeginChildren();
     while (i != cont->EndChildren()) {
-      AliEveTrack* track = dynamic_cast<AliEveTrack*>(*i);
+      AliEveTrack* track = static_cast<AliEveTrack*>(*i);
       map.insert(std::make_pair(track->GetLabel(), track));
       if (recurse)
         MapTracks(map, track, recurse);
@@ -61,7 +61,7 @@ void AliEveKineTools::SetDaughterPathMarks(TEveElement* cont, AliStack* stack, B
   TEveElement::List_i  iter = cont->BeginChildren();
   while(iter != cont->EndChildren())
   {
-    AliEveTrack* track = dynamic_cast<AliEveTrack*>(*iter);
+    AliEveTrack* track = static_cast<AliEveTrack*>(*iter);
     TParticle* p = stack->Particle(track->GetLabel());
     if (p->GetNDaughters())
     {
index 52b0792..79aadb9 100644 (file)
@@ -69,7 +69,7 @@ void AliEveKinkEditor::SetModel(TObject* obj)
 {
   // Set model object.
 
-  fM = dynamic_cast<AliEveKink*>(obj);
+  fM = static_cast<AliEveKink*>(obj);
 
   // Set values of widgets
   fInfoLabel0->SetText(Form("Radius = %f, Kink Angle = %f", fM->GetKinkRadius(), (fM->GetKinkAngle(2))*TMath::RadToDeg() ));
index 09c1a02..d68346f 100644 (file)
@@ -323,7 +323,7 @@ void AliEveITSScaledModule::LoadQuads()
        }
        else
        {
-         sd = dynamic_cast<ScaledDigit_t*>(GetId(miter->second));
+         sd = static_cast<ScaledDigit_t*>(GetId(miter->second));
          if(c1 < sd->fMinI)
            sd->fMinI = c1;
          else if( c1 > sd->fMaxI)
index b983442..58407eb 100644 (file)
@@ -398,7 +398,7 @@ void AliRun::SetRunLoader(AliRunLoader* rloader)
   TString evfoldname;
   TFolder* evfold = fRunLoader->GetEventFolder();
   if (evfold) evfoldname = evfold->GetName();
-  else AliWarning("Did not get Event Folder from Run Loader");
+  else AliFatal("Did not get Event Folder from Run Loader");
   
   if ( fRunLoader->GetAliRun() )
    {//if alrun already exists in folder