[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] vnc: tight: Fix crash after 2GB of output

From: Corentin Chary
Subject: Re: [Qemu-devel] [PATCH] vnc: tight: Fix crash after 2GB of output
Date: Fri, 4 Mar 2011 22:08:32 +0100

On Fri, Mar 4, 2011 at 12:46 PM, Michael Tokarev <address@hidden> wrote:
> 04.03.2011 11:56, Corentin Chary wrote:
>>>>     bytes = zstream->total_out - previous_out;
>> Good catch
>>> total_out isn't used by zlib internally, so if the resulting
>>> "total" counter is not needed in qemu, we can just zero-out
>>> the total_out in this function before calling zlib, and
>>> use the resulting value directly as "bytes", without
>>> saving its previous value in previous_out.  Something like
>>> the attached patch does.
>> If you're certain that total_out is not used by zlib, could you also
>> send a patch for zlib encoding please ? (vnc-enc-zlib.c)
>> Thanks,
> Yes, I noticed this too (the same code is in enc-zlib), and mentioned
> this in my previous email.
> The attached slightly different patch fixes both places and fixes
> them for good (hopefully anyway).  Runtime-tested for the tight
> case, but honestly, I didn't wait for 2G of output ;)
> Thanks!
> /mjt

Could you re-send it inline (not as an attachment), and CC Anthony ?

Corentin Chary

reply via email to

[Prev in Thread] Current Thread [Next in Thread]