Changeset 2785 for trunk/external-alma/atnf/PKSIO/ASTEDataset.cc

Timestamp:

03/08/13 17:58:53 (11 years ago)

Author:

Takeshi Nakazato

Message:

New Development: No

JIRA Issue: No

Ready for Test: Yes

Interface Changes: No

What Interface Changed: Please list interface changes

Test Programs: List test programs

Put in Release Notes: Yes/No?

Module(s): Module Names change impacts.

Description: Describe your changes here...

Candidate bug fix for seg. fault on Mac.

File:

• trunk/external-alma/atnf/PKSIO/ASTEDataset.cc (modified) (3 diffs)

trunk/external-alma/atnf/PKSIO/ASTEDataset.cc

@@ -35 +35 @@
 #include <iostream>
 #include <cstring>
+
+#define STRING2CHAR(s) const_cast<char *>((s).c_str())
 
 using namespace std ;
@@ -79 +81 @@
   int arymax = arrayMax();
 
-  string str120( 120, ' ' ) ;
-  char c120[120] ;
   // make sure file pointer points a beginning of the file
   fseek( fp_, 0, SEEK_SET ) ;
@@ -87 +87 @@
 
   // specific part
-  strcpy( c120, str120.c_str() ) ;
-  if ( readHeader( c120, 116 ) == -1 ) {
+  CDMY1.resize(116);
+  if ( readHeader( STRING2CHAR(CDMY1), 116 ) == -1 ) {
     os << LogIO::WARN << "Error while reading data CDMY1." << LogIO::POST ;
     return -1 ;
   }
-  c120[116] = '\0' ;
-  CDMY1 = string( c120 ) ;
   // DEBUG
   //cout << "CDMY1 = " << CDMY1 << endl ;