Boy...  on the one hand I hate to be putting in non-trivial changes this
close to freeze, but on the other hand you've clearly done some good
work chasing this down and it would be a shame to leave it out.

Normally, I'd want to see this incorporated into the upstream tree,
however at least  you've gotten the change reviewed by whot, and it
sounds like he didn't spot obvious flaws, just that he has larger plans
in motion.  I certainly don't mind papering over issues - anything to
make crashes go away!

I'd like to have seen it more widely tested as well, but your analysis
and testing sounds like it was done carefully.  I'll incorporate it,
provisionally that it is not implicated with any regressions.   We can
always revert the patch if we find problems with it.


** Changed in: xorg-server (Ubuntu)
   Importance: Undecided => Medium

** Changed in: xorg-server (Ubuntu)
       Status: New => Fix Committed

** Changed in: xorg-server (Ubuntu)
     Assignee: (unassigned) => Bryce Harrington (bryceharrington)

-- 
Server crash in miSpriteRestoreCursor()
https://bugs.launchpad.net/bugs/358009
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to