Coverity errors corrected/avoided.
authormorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sat, 9 Oct 2010 19:34:40 +0000 (19:34 +0000)
committermorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sat, 9 Oct 2010 19:34:40 +0000 (19:34 +0000)
ANALYSIS/AliAnalysisTaskESDfilter.cxx
STEER/AliAODEvent.cxx
STEER/AliAODHeader.h
STEER/AliESDEvent.cxx

index 57e32b0..6bb3c9c 100644 (file)
@@ -123,10 +123,8 @@ void AliAnalysisTaskESDfilter::ConvertESDtoAOD() {
     AliESD*      old = esd->GetAliESDOld();\r
 \r
     // Fetch Stack for debuggging if available \r
-    AliStack *pStack = 0;\r
     AliMCEventHandler *mcH = 0;\r
     if(MCEvent()){\r
-      pStack = MCEvent()->Stack();\r
       mcH = (AliMCEventHandler*) ((AliAnalysisManager::GetAnalysisManager())->GetMCtruthEventHandler()); \r
     }\r
     // set arrays and pointers\r
index 350f99f..e69001c 100644 (file)
@@ -484,7 +484,8 @@ Int_t AliAODEvent::GetMuonTracks(TRefArray *muonTracks) const
 
   AliAODTrack *track = 0;
   for (Int_t iTrack = 0; iTrack < GetNTracks(); iTrack++) {
-    if ((track = GetTrack(iTrack))->IsMuonTrack()) {
+    track = GetTrack(iTrack);
+    if (track->IsMuonTrack()) {
       muonTracks->Add(track);
     }
   }
index f9d50a3..44a5e98 100644 (file)
@@ -112,7 +112,8 @@ class AliAODHeader : public AliVHeader {
   void RemoveQTheta();
 
   void SetDiamond(Float_t xy[2],Float_t cov[3]) { 
-    for(Int_t i=0;i<3;i++) {if(i<2) fDiamondXY[i]=xy[i]; fDiamondCovXY[i]=cov[i];}
+    for(Int_t i=0;i<3;i++) {fDiamondCovXY[i] = cov[i];}
+    for(Int_t i=0;i<2;i++) {fDiamondXY[i]    = xy[i] ;}
   }
   void SetDiamondZ(Float_t z, Float_t sig2z){
     fDiamondZ=z; fDiamondSig2Z=sig2z;
index a0d8e3c..aa41831 100644 (file)
@@ -249,8 +249,11 @@ AliESDEvent & AliESDEvent::operator=(const AliESDEvent& source) {
        ((TNamed*)mine)->SetName(name);
       }
       else if(mine->InheritsFrom("TCollection")){
-       if(mine->InheritsFrom("TClonesArray"))
-         dynamic_cast<TClonesArray*>(mine)->SetClass(dynamic_cast<TClonesArray*>(its)->GetClass());
+       if(mine->InheritsFrom("TClonesArray")) {
+         TClonesArray* tcits = dynamic_cast<TClonesArray*>(its);
+         if (tcits)
+           dynamic_cast<TClonesArray*>(mine)->SetClass(tcits->GetClass());
+       }
        dynamic_cast<TCollection*>(mine)->SetName(name);
       }
       AliDebug(1, Form("adding object %s of type %s", mine->GetName(), mine->ClassName()));