summaryrefslogtreecommitdiff
path: root/libs/ui/GraphicBufferMapper.cpp
diff options
context:
space:
mode:
author Daniel Jarai <daniel.jarai@bartec-pixavi.com> 2017-09-20 11:33:51 +0200
committer Dániel Járai <daniel.jarai@bartec-pixavi.com> 2017-09-20 22:19:25 +0000
commite9147c2d7de146e8d98d58c37bc7ad55abb29dfd (patch)
treeda7d143f68c5b0736dd1e1ca411e1b35f82b6999 /libs/ui/GraphicBufferMapper.cpp
parent9a8ea13a26f8ddc1392943d606fd5282b1b33d87 (diff)
Avoid closing invalid fds in GraphicBufferMapper
unlockAsync will not necessarily return a valid FD. Change-Id: I86b06ec451017c6b665c545e2455883fdb8a3e47
Diffstat (limited to 'libs/ui/GraphicBufferMapper.cpp')
-rw-r--r--libs/ui/GraphicBufferMapper.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/ui/GraphicBufferMapper.cpp b/libs/ui/GraphicBufferMapper.cpp
index b9fa6400f5..5b0e7f6020 100644
--- a/libs/ui/GraphicBufferMapper.cpp
+++ b/libs/ui/GraphicBufferMapper.cpp
@@ -95,7 +95,7 @@ status_t GraphicBufferMapper::unlock(buffer_handle_t handle)
{
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);
}