Sorry to bother again, but ...

[Laurent Darrambide]

> 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;
> 

cd /tmp/bogofilter-0.94.2/src
cat patch-bogofilter | patch -p0

output:

patching file bogoutil.c
Hunk #1 succeeded at 798 with fuzz 1.
Hunk #2 FAILED at 818.
1 out of 2 hunks FAILED -- saving rejects to file bogoutil.c.rej

Why??

Attached: bogoutil.c.rej


-- 
Laurent Darrambide, France
GNU/Linux Debian Sid, kernel 2.6.10 - Mutt 1.5.6

-------------- next part --------------
***************
*** 817,823 ****
                 rc = ds_purgelogs(ds_file);
             break;
         case M_REMOVEENV:
-            ds_init(bogohome, ds_file);
             if (fTransaction)
                 rc = ds_remove(ds_file);
             break;
--- 818,823 ----
                 rc = ds_purgelogs(ds_file);
             break;
         case M_REMOVEENV:
             if (fTransaction)
                 rc = ds_remove(ds_file);
             break;
-------------- next part --------------
_______________________________________________
Bogofilter mailing list
Bogofilter at bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter