shairai / angleproject

Automatically exported from code.google.com/p/angleproject
Other
0 stars 0 forks source link

Framebuffer zero used after being destroyed in the Context destructor. #950

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
This is a small issue which shows up when I tried to use AppVerifier with ANGLE 
end2end.

When the context is destroyed, it needs to delete all the framebuffers. When a 
bound framebuffer is destroyed, it is detached of the read and draw binding 
points and the framebuffer zero is bound instead. Unfortunately the framebuffer 
zero is destroyed first hence the we try to attach a framebuffer which is 
already destroyed.

Original issue reported on code.google.com by Gregory....@imgtec.com on 17 Mar 2015 at 4:50

GoogleCodeExporter commented 9 years ago
https://chromium-review.googlesource.com/#/c/260573

Original comment by Gregory....@imgtec.com on 17 Mar 2015 at 5:01

GoogleCodeExporter commented 9 years ago
Project  : angle/angle
Branch   : master
Author   : Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>
Committer: Jamie Madill <jmadill@chromium.org>
Commit   : ed54e5db326cfd0c8db67f671b0c98834f8a6e46

Code-Review  0 : Gregoire Payen de La Garanderie
Code-Review  +1: Geoff Lang
Code-Review  +2: Jamie Madill
Verified     0 : Geoff Lang, Jamie Madill
Verified     +1: Gregoire Payen de La Garanderie
Commit Queue   : Chumped
Change-Id      : I06e9e6ccf92ee8ca0b1de98cc48ef68a85107214
Reviewed-at    : https://chromium-review.googlesource.com/260573

Fix framebuffer zero used after being destroyed in Context destructor.

BUG=angleproject:950

src/libANGLE/Context.cpp

Original comment by bugdro...@chromium.org on 19 Mar 2015 at 5:23

GoogleCodeExporter commented 9 years ago
Fixed, but assigning to me for a unit test when possible.

Original comment by jmad...@chromium.org on 19 Mar 2015 at 5:24