[PATCH] compr_lzo.c: allocate enough memory for lzo compressor.

Joakim Tjernlund joakim.tjernlund at transmode.se
Fri Apr 11 06:52:02 EDT 2008


On Fri, 2008-04-11 at 12:33 +0300, Riku Voipio wrote:
> This is the same bug as in kernel, pointed out the LZO author
> (Markus Oberhumer):
> 
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=f2a11b158a24301e9158e9c873fa88e5eb775486
> ---
>  compr_lzo.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/compr_lzo.c b/compr_lzo.c
> index fb54600..a0bb362 100644
> --- a/compr_lzo.c
> +++ b/compr_lzo.c
> @@ -97,7 +97,7 @@ int jffs2_lzo_init(void)
>  		return -1;
>  
>  	/* Worse case LZO compression size from their FAQ */
> -	lzo_compress_buf = malloc(page_size + (page_size / 64) + 16 + 3);
> +	lzo_compress_buf = malloc(page_size + (page_size / 16) + 64 + 3);
>  	if (!lzo_compress_buf) {
>  		free(lzo_mem);
>  		return -1;

I think you should use lzo1x_worst_compress() from the link you posted
above.

  Jocke



More information about the linux-mtd mailing list