tree 9916a67f5e0143c9bc647a899bf0495794729cda
parent 649bafb7ba2c3204295bc3451ff7af7e28e714ed
author Jesse Hall <jessehall@google.com> 1413914957 -0700
committer Jesse Hall <jessehall@google.com> 1413915113 -0700

surfaceflinger: don't close fence fds after passing to queueBuffer

ANativeWindow::queueBuffer takes ownership of the fence fd passed to
it, and will close it before returning. SurfaceFlinger's screenshot
code was also closing the syncFd it passed to queueBuffer. Most of the
time this meant the second close() silently failed, but in a rare race
condition the file descriptor could be reused between the two
close()s.

Bug: 17946343
Change-Id: Ib74fcb1dce52cc21328059c99b7c4c76f41aa3a5