Re: Feature: Fedora Electronic Lab

[Denis Leroy <denis poolshark org>]

Chitlesh GOORAH wrote:
> On 8/14/07, Hans de Goede wrote:
> > Same here, I can try to fix the segfault to, under pretty much the same
> > conditions as Kevin.

Calling free() on stuff allocated by Tcl_alloc(). bad bad bad.
Here's a patch.

-The Bug Squad

--- netgen-1.3.7/base/spice.c~	2006-04-11 00:04:49.000000000 +0200
+++ netgen-1.3.7/base/spice.c	2007-08-14 14:45:37.000000000 +0200
@@ -395,7 +395,7 @@
 
       strcpy(subcktname, scan->name);
       free (scan->name);
-      free (scan);
+      FREE (scan);
 
       /* nexttok is now NULL, scan->name points to class */
       Instance(subcktname, instancename);
--- netgen-1.3.7/base/netcmp.c~	2006-04-11 00:04:49.000000000 +0200
+++ netgen-1.3.7/base/netcmp.c	2007-08-14 14:30:16.000000000 +0200
@@ -1086,7 +1086,7 @@
     }
   }
 
-  free (LookupElementList);
+  FREE (LookupElementList);
 }
 
 #else