CarpetLib: Use vector<char> instead of new char in commstate class

Use vector<char> instead of new char[] in the commstate class.  This
corrects a memory management error.

Use .AT() instead of [] to access vector elements to catch indexing
errors.

darcs-hash:20080219044221-dae7b-ecd72b45833617920a33311953d5c2f00c42568c.gz

1 files changed, 3 insertions, 4 deletions

diff --git a/Carpet/CarpetLib/src/commstate.hh b/Carpet/CarpetLib/src/commstate.hh
index aac7e4f64..8bc3d0869 100644
--- a/Carpet/CarpetLib/src/commstate.hh
+++ b/Carpet/CarpetLib/src/commstate.hh
@@ -51,8 +51,8 @@ public:
   // structure describing a per-processor buffer for collective communications
   struct procbufdesc {
     // the allocated communication buffers
-    char* sendbufbase;
-    char* recvbufbase;
+    vector<char> sendbufbase;
+    vector<char> recvbufbase;
 
     // the sizes of communication buffers (in elements of type <datatype>)
     size_t sendbufsize;
@@ -64,8 +64,7 @@ public:
     char* recvbuf;
 
     // constructor for an instance of this structure
-    procbufdesc() : sendbufbase(NULL), recvbufbase(NULL),
-                    sendbufsize(0), recvbufsize(0),
+    procbufdesc() : sendbufsize(0), recvbufsize(0),
                     sendbuf(NULL), recvbuf(NULL)
     {
     }