Sorry to bother again, but ...

[David Relson]

On Fri, 25 Mar 2005 09:32:42 +0100
Laurent Darrambide wrote:

> 
> Hi everyone,
> 
> I upgraded today from 0.94.1 to 0.94.2 this morning on my Debian Sid PC.
> 
> Here we go again:
> 
> 
> bogoutil --db-verify wordlist.db 
> Erreur de segmentation (seg fault)
>  
> bogoutil -d wordlist.db | bogoutil -l wordlist.db.new
> Lock table is out of available locks
> bogoutil: (db) DB->get(TXN=134699160,  'whirled' ), err: Cannot allocate
> memory
> 
> 
> back to 0.94.1 everything's fine.
> 
> Aaarrrgghhh.

Laurent,

Below is a patch so bogoutil that fixes the sigsegv when --db-verify is
used in a non-transactional environment.  It makes the option a
"nop" (do nothing), rather than a crash.  Since non-transactional
databases lack the log files needed for recovery, a nop may be the best
that we can do.

Regards,

David

--- bogoutil.c	17 Mar 2005 12:35:13 -0000	1.200
+++ bogoutil.c	25 Mar 2005 23:26:38 -0000
@@ -798,6 +798,7 @@
     }
 
     set_bogohome(ds_file);
+    ds_init(bogohome, ds_file);
 
     switch(flag) {
 	case M_RECOVER:
@@ -817,7 +818,6 @@
 		rc = ds_purgelogs(ds_file);
 	    break;
 	case M_REMOVEENV:
-	    ds_init(bogohome, ds_file);
 	    if (fTransaction)
 		rc = ds_remove(ds_file);
 	    break;
_______________________________________________
Bogofilter mailing list
Bogofilter at bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter