use macro to define name string of the log
authorGengbin Zheng <gzheng@illinois.edu>
Thu, 7 Apr 2011 15:06:58 +0000 (10:06 -0500)
committerGengbin Zheng <gzheng@illinois.edu>
Thu, 7 Apr 2011 15:06:58 +0000 (10:06 -0500)
src/langs/bluegene/bigsim_logs.C
src/langs/bluegene/bigsim_logs.h

index 125a6d6f0c62de00c82dc460c5f52bd96e074cff..aa816175b3a329dcb50a60754b9f5fed99641039 100644 (file)
@@ -158,7 +158,8 @@ BgTimeLog::BgTimeLog()
 
 BgTimeLog::BgTimeLog(BgTimeLog *log)
 {
-  strncpy(name,log->name,20);
+  strncpy(name,log->name,BGLOG_NAMELEN-1);
+  name[BGLOG_NAMELEN-1] = 0;
   ep = log->ep;
   charm_ep = -1;
   startTime = log->startTime;
@@ -197,7 +198,8 @@ BgTimeLog::BgTimeLog(int epc, const char* namestr,double sTime)
 { 
   if(namestr == NULL)
     namestr = (char*)"dummyname1";
-  strncpy(name,namestr,20);
+  strncpy(name,namestr,BGLOG_NAMELEN-1);
+  name[BGLOG_NAMELEN-1] = 0;
   ep = epc;
   charm_ep = -1;
   startTime = sTime;
@@ -217,7 +219,8 @@ BgTimeLog::BgTimeLog(int epc, const char* namestr, double sTime, double eTime)
 {
   if(namestr == NULL)
     namestr = (char*)"dummyname2";
-  strncpy(name,namestr, 20);
+  strncpy(name,namestr, BGLOG_NAMELEN-1);
+  name[BGLOG_NAMELEN-1] = 0;
   ep = epc;
   charm_ep = -1;
   startTime = sTime;
@@ -407,7 +410,7 @@ void BgTimeLog::pupCommon(PUP::er &p) {
 
   if (bglog_version >= 4) p(charm_ep);
   p|recvTime; p|effRecvTime; p|startTime; p|execTime; p|endTime; 
-  p|flag; p(name,20);
+  p|flag; p(name,BGLOG_NAMELEN);
   if (bglog_version >= 3)
     p((int *)&objId, sizeof(CmiObjId)/sizeof(int));
   else if (bglog_version == 2)
index bfcc32816460a0e0f3d3bedcbd707471f3dc99e3..1a07e35a0c7466b30b180d772c1a764bd87cdc52 100644 (file)
@@ -113,6 +113,8 @@ class BgTimeLineRec;
 
 enum BgMPIOp { MPI_NONE = 0, MPI_BARRIER = 1, MPI_ALLREDUCE = 2 };
 
+#define BGLOG_NAMELEN   20
+
 /**
   one time log for an handler function;
   it records a list of message sent events in an execution of handler
@@ -137,7 +139,7 @@ public:
   unsigned short mpiOp;
   short charm_ep;
 
-  char name[20];
+  char name[BGLOG_NAMELEN];
   char doCorrect;
   char flag;