PATCH: hisv6_replace() fix

Chris Caputo ccaputo at alt.net
Mon Jun 6 21:31:07 UTC 2005


Oops.  Hold off on this patch.  Found another issue and will have better
patch shortly.

Chris

On Mon, 6 Jun 2005, Chris Caputo wrote:
> Hi.  Simple fix for hisv6_replace()...  (pointers instead of lengths were
> being checked)
> 
> Below and at:
> 
>    http://www.caputo.com/foss/inn-2.4.2-hisv6_replacefix.patch
> 
> Thanks,
> Chris
> 
> ------
> 
> --- inn-2.4.2-stock/history/hisv6/hisv6.c	2004-12-22 04:21:19.000000000 +0000
> +++ inn-2.4.2/history/hisv6/hisv6.c	2005-06-06 19:07:38.365319315 +0000
> @@ -924,7 +924,7 @@ hisv6_replace(void *history, const char 
>  
>  	    oldlen = strlen(old);
>  	    newlen = strlen(new);
> -	    if (new > old) {
> +	    if (newlen > oldlen) {
>  		hisv6_seterror(h, concat("new history line too long ",
>  					  h->histpath, NULL));
>  		r = false;
> 
> 



More information about the inn-workers mailing list