qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [7022] vnc: Fix incorrect attempt to clear a flag
@ 2009-04-07 19:57 Aurelien Jarno
  0 siblings, 0 replies; only message in thread
From: Aurelien Jarno @ 2009-04-07 19:57 UTC (permalink / raw)
  To: qemu-devel

Revision: 7022
          http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=7022
Author:   aurel32
Date:     2009-04-07 19:57:09 +0000 (Tue, 07 Apr 2009)
Log Message:
-----------
vnc: Fix incorrect attempt to clear a flag

In vnc.c in pixel_format_message, the code tries to clear the
QEMU_ALLOCATED_FLAG from the client display surface, however
it uses the wrong operator and ends up enabling all other
flags. Most notably this enables the big endian flag and
causes some chaos.

Signed-off-by: Brian Kress <kressb@moose.net>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

Modified Paths:
--------------
    trunk/vnc.c

Modified: trunk/vnc.c
===================================================================
--- trunk/vnc.c	2009-04-07 18:43:28 UTC (rev 7021)
+++ trunk/vnc.c	2009-04-07 19:57:09 UTC (rev 7022)
@@ -1598,7 +1598,7 @@
     else if (vs->ds->surface->pf.bits_per_pixel == 8)
         vs->send_hextile_tile = send_hextile_tile_8;
     vs->clientds = *(vs->ds->surface);
-    vs->clientds.flags |= ~QEMU_ALLOCATED_FLAG;
+    vs->clientds.flags &= ~QEMU_ALLOCATED_FLAG;
     vs->write_pixels = vnc_write_pixels_copy;
 
     vnc_write(vs, pad, 3);           /* padding */

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2009-04-07 19:57 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-04-07 19:57 [Qemu-devel] [7022] vnc: Fix incorrect attempt to clear a flag Aurelien Jarno

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).