Comment 19 for bug 358009

Revision history for this message
Tom Jaeger (thjaeger) wrote : Re: [Bug 358009] Re: Server crash in miSpriteRestoreCursor()

Thank you.

Bryce Harrington wrote:
> 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)
>