ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add lock before calling initEglImage

Browse Source 
Without that lock, there is a chance of race condition
where while composing a specific index, requestBuf with
the same index can be executed and touch the
same data that is being used in initEglImage.
(e.g. dirty flag in texture)
This commit is contained in:
Kobi Cohen Arazi 2011-04-15 10:38:33 -07:00
parent d59b5d0087
commit 0d11baf889
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
services/surfaceflinger/Layer.cpp
View File

@ -874,8 +874,16 @@ status_t Layer::BufferManager::initEglImage(EGLDisplay dpy,
ssize_t index = mActiveBuffer; ssize_t index = mActiveBuffer;
if (index >= 0) { if (index >= 0) {
if (!mFailover) { if (!mFailover) {
Image& texture(mBufferData[index].texture); {
err = mTextureManager.initEglImage(&texture, dpy, buffer); // Without that lock, there is a chance of race condition
// where while composing a specific index, requestBuf
// with the same index can be executed and touch the same data
// that is being used in initEglImage.
// (e.g. dirty flag in texture)
Mutex::Autolock _l(mLock);
Image& texture(mBufferData[index].texture);
err = mTextureManager.initEglImage(&texture, dpy, buffer);
}
// if EGLImage fails, we switch to regular texture mode, and we // if EGLImage fails, we switch to regular texture mode, and we
// free all resources associated with using EGLImages. // free all resources associated with using EGLImages.
if (err == NO_ERROR) { if (err == NO_ERROR) {