[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:56:09 +0000 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Tue, 22 Nov 2011, Markus Armbruster wrote:
> Stefano Stabellini <address@hidden> writes:
>
> > 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).
>
> I'll post the obvious patch then. Thanks!
you are welcome