[vbox-dev] A Chromium bug in state_client.c
haitao.feng at intel.com
Tue Aug 10 00:00:25 GMT 2010
Thank you for the reviewing and merging!
Yes, those lines are not needed. I should have reviewed this patch more times.
Could I know your plan to run Mutter with GL acceleration?
On Mon, Aug 09, 2010 at 10:09:42PM +0800, Leonid wrote:
> Hi Haitao,
> > Hi,
> > Recently I have found a bug in
> > src/VBox/GuestHost/OpenGL/state_tracker/state_client.c.
> > When there are GL client states changing, the dirty bit is fogetten to be set.
> > This means those GL state could not be changed back in the future. I found this
> > bug when I am enabling VirtualBox to run MeeGo Netbook UX which is a mutter
> > plugin.
> > The code patch is attached. I have read your "How to contribute" page and want
> > to license this patch under MIT.
> > Thanks
> > -Haitao
> > Attachment (state_client.c.patch): text/x-diff, 6070 bytes
> Thanks for a finding. Your patch is partially in the tree now, with the
> exception of lines like "FILLDIRTY(cb->clientPointer);" as those shouldn't be
> needed under "if (CHECKDIRTY(cb->enableClientState, bitID))".
> Please let me know if you could test it in your setup. If those lines are
> required to workaround the issue, then they're hiding a bug in some other place.
> Thanks, Leonid.
> vbox-dev mailing list
> vbox-dev at virtualbox.org
More information about the vbox-dev