commit e9147c2d7de146e8d98d58c37bc7ad55abb29dfd
author Daniel Jarai <daniel.jarai@bartec-pixavi.com> Wed Sep 20 11:33:51 2017 +0200
committer Dániel Járai <daniel.jarai@bartec-pixavi.com> Wed Sep 20 22:19:25 2017 +0000
tree da7d143f68c5b0736dd1e1ca411e1b35f82b6999
parent 9a8ea13a26f8ddc1392943d606fd5282b1b33d87

Avoid closing invalid fds in GraphicBufferMapper

unlockAsync will not necessarily return a valid FD.

Change-Id: I86b06ec451017c6b665c545e2455883fdb8a3e47

libs/ui/GraphicBufferMapper.cpp

diff --git a/libs/ui/GraphicBufferMapper.cpp b/libs/ui/GraphicBufferMapper.cpp
index b9fa640..5b0e7f6 100644
--- a/libs/ui/GraphicBufferMapper.cpp
+++ b/libs/ui/GraphicBufferMapper.cpp
@@ -95,7 +95,7 @@
 {
     int32_t fenceFd = -1;
     status_t error = unlockAsync(handle, &fenceFd);
-    if (error == NO_ERROR) {
+    if (error == NO_ERROR && fenceFd >= 0) {
         sync_wait(fenceFd, -1);
         close(fenceFd);
     }