4c05dd175e
onInitializeDisplays() was posting a transaction with changes to the display projection. Unfortunately, it only set the display orientation field and left viewport and frame uninitialized. The uninitialized values flowed downstream and found themselves baked into a bogus DisplayDevice mGlobalTransform. That transform was then applied to some Rects which were turned into Regions that were them combined with other Regions. Under certain situations, the uninitialized data might have a largish value, resulting in the creation of Regions with components in excess of the Region max-value limit of 0x7ffffff (note that this is not INT_MAX). Later when performing a binary operation using the Region, the Spanner would loop indefinitely trying to figure out how to stuff a humongous region inside of a max-value region. Not content to try just once, the Spanner would continue trying again and again, pegging the CPU and hanging surface flinger during boot. Insanity soon followed. Bug: 7130713 Change-Id: I0016f0c9662185be833474c212a1dd408096ae23 |
||
---|---|---|
.. | ||
DisplayHardware | ||
tests | ||
Android.mk | ||
Barrier.h | ||
Client.cpp | ||
Client.h | ||
clz.cpp | ||
clz.h | ||
DdmConnection.cpp | ||
DdmConnection.h | ||
DisplayDevice.cpp | ||
DisplayDevice.h | ||
EventThread.cpp | ||
EventThread.h | ||
GLExtensions.cpp | ||
GLExtensions.h | ||
Layer.cpp | ||
Layer.h | ||
LayerBase.cpp | ||
LayerBase.h | ||
LayerDim.cpp | ||
LayerDim.h | ||
LayerScreenshot.cpp | ||
LayerScreenshot.h | ||
MessageQueue.cpp | ||
MessageQueue.h | ||
MODULE_LICENSE_APACHE2 | ||
SurfaceFlinger.cpp | ||
SurfaceFlinger.h | ||
SurfaceTextureLayer.cpp | ||
SurfaceTextureLayer.h | ||
Transform.cpp | ||
Transform.h |