Changeset 603a989 in git

Timestamp:

Jul 20, 2018, 11:37:45 AM (5 years ago)

Author:

Hans Schoenemann <hannes@…>

Branches:

(u'spielwiese', '8e0ad00ce244dfd0756200662572aef8402f13d5')

Children:

432937cbb1c75e4f468b7a2502e94d94d86528e2

Parents:

f4f135ee4889c610adf6c8d7ccea7e4e95f97b2a

Message:

fix: prune for "memory leak in factorize (tmp. extensions)"

File:

• factory/variable.cc (modified) (2 diffs)

factory/variable.cc

@@ -264 +264 @@
   int last_var=-alpha.level();
   if ((last_var <= 0)||(var_names_ext==NULL)) return;
+  int i, n = strlen( var_names_ext );
+  ASSERT (n+1 >= last_var, "wrong variable");
   if (last_var == 1)
   {
@@ -273 +275 @@
     return;
   }
-  ASSERT (strlen(var_names_ext)+1 >= -alpha.level(), "wrong variable");
   char * newvarnames = new char [last_var+1];
-  ext_entry * newalgext = new ext_entry [last_var];
-  for (int i = 0; i < last_var; i++ )
-  {
+  for ( i = 0; i < last_var; i++ )
     newvarnames[i] = var_names_ext[i];
-    newalgext[i] = algextensions[i];
-  }
   newvarnames[last_var] = 0;
   delete [] var_names_ext;
   var_names_ext = newvarnames;
+  ext_entry * newalgext = new ext_entry [last_var];
+  for ( i = 0; i < last_var; i++ )
+    newalgext[i] = algextensions[i];
   delete [] algextensions;
   algextensions = newalgext;