Merge pull request #137 from ampli/setup_domain_array

Fix memory reallocation in setup_domain_array
opencog · Jul 8, 2015 · 4d7205f · 4d7205f
2 parents 0b83268 + 6efc3e8
commit 4d7205f
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/link-grammar/post-process.c b/link-grammar/post-process.c
@@ -610,14 +610,14 @@ static void setup_domain_array(Postprocessor *pp,
 	/* Grab more memory if needed */
 	if (pp->pp_data.domlen <= n)
 	{
-		size_t newsz, incsz;
+		size_t oldsz, incsz;
 #define DOMINC 16
-		pp->pp_data.domlen += DOMINC;
-		newsz = pp->pp_data.domlen * sizeof(Domain);
+		oldsz = pp->pp_data.domlen * sizeof(Domain);
 		incsz = DOMINC * sizeof(Domain);
 		pp->pp_data.domain_array = (Domain *) realloc(pp->pp_data.domain_array,
-			newsz);
-		memset(&pp->pp_data.domain_array[n], 0, incsz);
+			oldsz + incsz);
+		memset(&pp->pp_data.domain_array[pp->pp_data.domlen], 0, incsz);
+		pp->pp_data.domlen += DOMINC;
 	}
 
 	pp->pp_data.domain_array[n].string = string;