[Spice-devel] Question on fill_mask() (server/red_worker.c)

Alon Levy alevy at redhat.com
Tue Oct 4 04:00:47 PDT 2011


On Tue, Oct 04, 2011 at 12:40:21PM +0200, Yaniv Kaul wrote:
> Unless I'm misreading the code (which may very well be), the
> function looks like:
> 
> if (mask_bitmap && m) {
>     if (this or that) {
>         do X
>         fill_bits(...)
>     } else {
>         fill_bits(...)
>     }
> }
> 
> So essentially, if the condition (mask_bitmap && m) is NOT met, we
> do not fill the bits. Which means we are sending garbage over the
> wire?

I read it the same. Can you add the else and see if it gets there? (or
figure that out by just reading the code)

> Y.
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel


More information about the Spice-devel mailing list