From e9147c2d7de146e8d98d58c37bc7ad55abb29dfd Mon Sep 17 00:00:00 2001
From: Daniel Jarai <daniel.jarai@bartec-pixavi.com>
Date: Wed, 20 Sep 2017 11:33:51 +0200
Subject: Avoid closing invalid fds in GraphicBufferMapper

unlockAsync will not necessarily return a valid FD.

Change-Id: I86b06ec451017c6b665c545e2455883fdb8a3e47
---
 libs/ui/GraphicBufferMapper.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'libs/ui/GraphicBufferMapper.cpp')

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);
     }
-- 
cgit v1.2.3-59-g8ed1b