- fixing coverity
authorodjuvsla <odjuvsla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Mar 2011 16:52:09 +0000 (16:52 +0000)
committerodjuvsla <odjuvsla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Mar 2011 16:52:09 +0000 (16:52 +0000)
HLT/CALO/AliHLTCaloMapper.cxx
HLT/CALO/AliHLTCaloProcessor.cxx
HLT/CALO/AliHLTCaloSharedMemoryInterfacev2.cxx

index 05fa055..65773bc 100644 (file)
@@ -55,6 +55,7 @@ AliHLTCaloMapper::AliHLTCaloMapper( const unsigned long  specification , TString
   fSpecificationMapPtr(0)
 {  
   //see header file for class documentation
+  fFilepath[0] = '\0';
 }
 
 
index c47c58a..31da68e 100644 (file)
@@ -28,9 +28,12 @@ AliHLTCaloProcessor::AliHLTCaloProcessor():AliHLTProcessor(),
                                           fModuleID(2), 
                                           fPrintInfoModule(0), 
                                           fPrintInfoFrequncyModule(1000), 
-                                          fRunNumber(0)
+                                          fRunNumber(0) 
 {
   //  ScanRunNumberFromFile();
+  lineNumber[0] = '\0';
+  fFilepath[0] = '\0';
+  fMessage[0] = '\0';
 }
 
 
@@ -42,41 +45,41 @@ AliHLTCaloProcessor::~AliHLTCaloProcessor()
 
  
 bool 
-AliHLTCaloProcessor::CheckFileLog(const char *origin,   const char *filename,  const char *opt)
+AliHLTCaloProcessor::CheckFileLog(const char */*origin*/,   const char */*filename*/,  const char */*opt*/)
 {
-  sprintf(fFilepath, "%s/%s", getenv("PWD"), filename);
-  FILE *fp = fopen(filename, opt);
-
-  if(fp == 0)
-    {
-      //      if( (opt == "w")  || (opt == "a")) \\OD
-      if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
-       {
-         sprintf(fMessage, "for writing  please check that the directory exists and that you have write access to it"  );
-       }
-      else
-       {
-         sprintf(fMessage, "for reading  please check that the directory exists and that you have read access to it"  );
-       }
-     Logging(kHLTLogFatal, origin , "cannot open file" , "Was not able to open file %s  %s", fFilepath, fMessage);
-     return false;
-    }
-  else
-    {
-      //      if( (opt == "w")  || (opt == "a")) \\OD
-      if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
-       {
-         sprintf(fMessage, "for writing" );
-       }
-      else
-       {
-         sprintf(fMessage, "for reading");
-       }
-      //    Logging(kHLTLogInfo, origin , "opening file" , "Sucessfully opening %s  %s", fFilepath, fMessage);
-      fclose(fp); 
-      return true;
-    }
-  
+  // sprintf(fFilepath, "%s/%s", getenv("PWD"), filename);
+  // FILE *fp = fopen(filename, opt);
+
+  // if(fp == 0)
+  //   {
+  //     //      if( (opt == "w")  || (opt == "a")) \\OD
+  //     if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
+  //   {
+  //     sprintf(fMessage, "for writing  please check that the directory exists and that you have write access to it"  );
+  //   }
+  //     else
+  //   {
+  //     sprintf(fMessage, "for reading  please check that the directory exists and that you have read access to it"  );
+  //   }
+  //    Logging(kHLTLogFatal, origin , "cannot open file" , "Was not able to open file %s  %s", fFilepath, fMessage);
+  //    return false;
+  //   }
+  // else
+  //   {
+  //     //      if( (opt == "w")  || (opt == "a")) \\OD
+  //     if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
+  //   {
+  //     sprintf(fMessage, "for writing" );
+  //   }
+  //     else
+  //   {
+  //     sprintf(fMessage, "for reading");
+  //   }
+  //     //    Logging(kHLTLogInfo, origin , "opening file" , "Sucessfully opening %s  %s", fFilepath, fMessage);
+  //     fclose(fp); 
+  //     return true;
+  //   }
+  return false;
 }
 
  
index b491069..26c1bff 100644 (file)
@@ -40,7 +40,10 @@ AliHLTCaloSharedMemoryInterfacev2::AliHLTCaloSharedMemoryInterfacev2(TString det
                                                                        //      fSpecification(0)
 {
    // See header file for class documentation
-   
+  for(Int_t i = 0; i < 32; i++)
+    {
+      fMapperPtr[i] = 0;
+    }
 }
 
 AliHLTCaloSharedMemoryInterfacev2::~AliHLTCaloSharedMemoryInterfacev2()