Comment 47 for bug 328035

Revision history for this message
Tormod Volden (tormodvolden) wrote :

I got suspicious when I saw LogVMessageVerb in the backtrace, which I have touched recently myself... Sure enough I did not think about the possibility of clocks going backwards (which they sometimes do after resume) when I worked on the timestamp stuff and you end up with nice timestamps (like below) with no bounds checking. The tmpBuf is overflowed and free()'ing it messes up. Sorry.

[-1009303149.602078] (II) AIGLX: Suspending AIGLX clients for VT switch

The timestamp patch was supposed to be taken away before Beta anyway. Bryce, can you please remove it now, and I will fix the timestamp again for later.