[Refdb-users] spoke too soon!

[Markus H. <mar...@mh...>]

Matt Price writes:
 > hey markus, the segfault and the free() error are back when i try to
 > addnote...  
 > 
 > sorry to report that...
 > matt

Turns out this was one of these bugs that are silent on FreeBSD. I
managed to reproduce the problem on Debian. Stupid me! The following
patch fixes the problem. I've also updated the file in CVS.

BTW: so much about temporary hacks. They've been in place for three
years...


--- refdbc.c.orig       Wed Feb 11 23:08:15 2004
+++ refdbc.c    Wed Feb 11 23:08:41 2004
@@ -3507,7 +3507,7 @@
   }
 
   /* the following is a temporary hack to allow cmdln_tokenize to work */
-  newarg = (char*)malloc((size_t)(strlen(arg)+8));
+  newarg = (char*)malloc((size_t)(strlen(arg)+9));
   if (newarg == NULL) {
     delete_all_lilimem(&sentinel);
     return 1;
@@ -4399,7 +4399,7 @@
   }
   
   /* the following is a temporary hack to allow cmdln_tokenize to work */
-  newarg = (char*)malloc((size_t)(strlen(arg)+8));
+  newarg = (char*)malloc((size_t)(strlen(arg)+9));
   if (newarg == NULL) {
     delete_all_lilimem(&sentinel);
     return 1;

regards,
Markus

-- 
Markus Hoenicka
mar...@ca...
(Spam-protected email: replace the quadrupeds with "mhoenicka")
http://www.mhoenicka.de