vlistCopy: allocate and copy varsAllocated elements (bug fix)

6055003a · Uwe Schulzweida · bde5460d · 6055003a · 6055003a
Commit 6055003a authored Jul 07, 2011 by Uwe Schulzweida
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

ChangeLog ChangeLog +4 -0

src/vlist.c src/vlist.c +2 -2

No files found.
--- a/ChangeLog
+++ b/ChangeLog
+2011-07-07  Uwe Schulzweida  <Uwe.Schulzweida@zmaw.de>
+
+	* vlistCopy: allocate and copy varsAllocated elements (bug fix) [report: Ralf Mueller]
+
 2011-06-24  Uwe Schulzweida  <Uwe.Schulzweida@zmaw.de>

 	* vtime2timeval: do not round result (bug fix for TUNIT_YEAR) [report: Andy Aschwanden]

--- a/src/vlist.c
+++ b/src/vlist.c
@@ -372,8 +372,8 @@ void vlistCopy(int vlistID2, int vlistID1)
      int nvars = vlistptr1->nvars;
      int nlevs, varID;

-      vlistptr2->vars = (var_t *) malloc(nvars*sizeof(var_t));
-      memcpy(vlistptr2->vars, vlistptr1->vars, nvars*sizeof(var_t));
+      vlistptr2->vars = (var_t *) malloc(vlistptr2->varsAllocated*sizeof(var_t));
+      memcpy(vlistptr2->vars, vlistptr1->vars, vlistptr2->varsAllocated*sizeof(var_t));

      for ( varID = 0; varID < nvars; varID++ )
 	{