[erlang-questions] zlib deflate problem

Sverker Eriksson sverker@REDACTED
Wed May 21 09:55:53 CEST 2008


Matthew Dempsky wrote:
> On Tue, May 20, 2008 at 10:43 AM, Colm Dougan <colm.dougan@REDACTED> wrote:
>   
>> Matthew - I tried your patch and it worked.   I don't see any
>> regression but I haven't done extensive testing. If you think this is
>> the appropriate fix (rather than an exploratory patch) then I can
>> install the patched erlang on my running system and do more testing.
>>     
>
> I think it should be safe for more testing.
>
>   
Hi guys

I've been looking at your patch Matthew, trying to understand it to 
decide if it should be part of the next OTP release. Do you have any 
more arguments why this would be the right way to do it?

> --- zlib_drv.c.orig	2008-05-19 19:09:00.000000000 -0700
> +++ zlib_drv.c	2008-05-19 19:09:12.000000000 -0700
> @@ -257,6 +257,10 @@
>  		driver_deq(d->port, len);
>  		return res;
>  	    }
> +	    if (res == Z_BUF_ERROR) {
> +		res = Z_OK;
> +	    }
>  	    if (res < 0) {
>  		return res;
>  	    }

As I understand it, a return of Z_BUF_ERROR from inflate() means 
something like "call me again with more buffer space". But in this case 
we don't call again.

Do you have any more light to shed?

/Sverker, Erlang/OTP Ericsson




More information about the erlang-questions mailing list