Closed
Bug 455423
Opened 16 years ago
Closed 15 years ago
Painting error with -moz-transform rotate and large translate
Categories
(Core :: Web Painting, defect)
Core
Web Painting
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: martijn.martijn, Unassigned)
References
Details
(Keywords: testcase, Whiteboard: [sg:moderate?])
Attachments
(4 files)
See testcase, when I click on the rotated textarea, it disappears (because of failed repainting or something like that). You can get the same painting bug, by moving a window over it.
Reporter | ||
Comment 1•16 years ago
|
||
Another case, in this case, the iframe becomes black.
Comment 2•16 years ago
|
||
Confirmed on Linux, on both testcases, via dragging another window over them.
(clicking the testcases doesn't seem to trigger the bug on testcase 2, and it only occasionally triggers the bug on testcase 1)
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b1pre) Gecko/20080914020418 Minefield/3.1b1pre
OS/Platform --> All/All
OS: Windows XP → All
Hardware: PC → All
Comment 3•16 years ago
|
||
Tentatively marking security-sensitive, as it looks like this might be reading uninitialized memory. (see upcoming screenshot)
Group: core-security
Comment 4•16 years ago
|
||
The patterns in this looks like uninitialized memory to me.
Reporter | ||
Comment 5•16 years ago
|
||
I can't reproduce that screenshot, I'm more or less getting a normal painting error.
There is no Firefox release yet out there, that has the -moz-transform feature in it.
That's basically my criterium to file a crash bug as security sensitive or not. If it's not crashing on branch, it's not security sensitive.
OS: All → Windows XP
Hardware: All → PC
Comment 6•16 years ago
|
||
(In reply to comment #5)
> I can't reproduce that screenshot, I'm more or less getting a normal painting
> error.
Yeah -- that could be just because the memory we read is platform-dependent. (i.e. on windows, it hits an area that's initialized to 0's, but on Linux, it hits in-use memory)
> There is no Firefox release yet out there, that has the -moz-transform feature
> in it.
Firefox 3.1b1 is coming up, though, and it's feasible that this bug won't be fixed by then, so I think it's better to initially leave this security-sensitive to protect beta-users.
Setting as "sg:moderate?" after discussion with dveditz.
OS: Windows XP → All
Hardware: PC → All
Whiteboard: [sg:moderate?]
Comment 7•16 years ago
|
||
Here's testcase 2 with a larger iframe, so more corruption is visible at once.
Playing around with this has shown that the corruption is mostly showing bits and pieces of other things on my screen. (i.e. pieces of other windows, files on my desktop, etc., *including* things I can't currently see because they're behind other windows.)
Comment 8•16 years ago
|
||
In the security review for the transform property we brought up issues with floating point overflows and it seems like that might be what's happening here. There's some code in nsStyleTransformMatrix that computes matrix products and I don't think that I did any bounds-checking on the resulting floats and nscoords. Would constraining these values to a certain range eliminate the problem?
Comment 9•15 years ago
|
||
These testcases all work for me on a current Linux trunk debug build. Do they still show problems for you?
Reporter | ||
Comment 10•15 years ago
|
||
Yes, they all seem to be wfm. Does this bug still need to be security sensitive?
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WORKSFORME
Comment 11•15 years ago
|
||
WFM too, with today's nightly...
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.3a4pre) Gecko/20100405 Minefield/3.7a4pre
...and with a nightly from when this was initially filed...
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b1pre) Gecko/20080915020438 Minefield/3.1b1pre
(currently using Ubuntu 10.04 b1 up-to-date)
I tried to reproduce with Compiz on and off, to see if that affected the old nightly's behavior, but I still couldn't repro. I wonder if that nightly becoming WFM is due to some Ubuntu system library changing?
(In reply to comment #10)
> Does this bug still need to be security sensitive?
Hm, would you mind checking if it's reproducible on Windows with a nightly from around the bug-file-date? If it still is, I'd imagine we'd want to double-check that 3.5 & 3.6 branches are unaffected before opening this up.
Updated•9 years ago
|
Group: core-security → core-security-release
Updated•9 years ago
|
Group: core-security-release
Assignee | ||
Updated•6 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•