[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Suspicious fall through in qemu_default_pixelformat()
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] Suspicious fall through in qemu_default_pixelformat() |
Date: |
Tue, 22 Nov 2011 11:30:12 +0000 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Tue, 22 Nov 2011, Markus Armbruster wrote:
> Case 24 falls through to case 32, which overwrites all of case 24's
> work. Looks wrong. Git blames commit 0da2ea1b "fix endianness problem
> sharing the videoram buffer".
>
> Patching in a break is easy enough, but since I don't actually
> understand how the function is used, I have no idea what exactly this
> fixes. Stefano, can you help?
You are right, there should be a break statement there.
However it doesn't matter in practice because the 24 and 32 bit settings
are the same (amask, ashift and abits don't matter much).