commit f53b377bde70eb4a28acb27713c17a855ef42495
author Naseer Ahmed <naseer@codeaurora.org> Fri Feb 15 19:13:50 2013 -0500
committer Naseer Ahmed <naseer@codeaurora.org> Fri Feb 15 19:13:50 2013 -0500
tree 3bf1d150ee8bcf3000326ff2c427ebad2fd21c57
parent 91dae25d49c3da2f628ee5b0ca4a1e86b31eaefd

display: Close 0 fds

0 can be a valid file descriptor, make sure 0 fds are closed

Change-Id: I2bf2280d70eda74fa0b0a886ea54ecefc6854b24

libexternal/external.cpp

diff --git a/libexternal/external.cpp b/libexternal/external.cpp
index 1c8f170..883a39f 100644
--- a/libexternal/external.cpp
+++ b/libexternal/external.cpp
@@ -450,7 +450,7 @@
 bool ExternalDisplay::closeFrameBuffer()
 {
     int ret = 0;
-    if(mFd > 0) {
+    if(mFd >= 0) {
         ret = close(mFd);
         mFd = -1;
     }

libhwcomposer/hwc_utils.cpp

diff --git a/libhwcomposer/hwc_utils.cpp b/libhwcomposer/hwc_utils.cpp
index 093c0b3..a011f9b 100644
--- a/libhwcomposer/hwc_utils.cpp
+++ b/libhwcomposer/hwc_utils.cpp
@@ -390,11 +390,11 @@
         if(list->hwLayers[i].compositionType == HWC_OVERLAY ||
            list->hwLayers[i].compositionType == HWC_FRAMEBUFFER_TARGET) {
             //Close the acquireFenceFds
-            if(list->hwLayers[i].acquireFenceFd > 0) {
+            if(list->hwLayers[i].acquireFenceFd >= 0) {
                 close(list->hwLayers[i].acquireFenceFd);
                 list->hwLayers[i].acquireFenceFd = -1;
             }
-            if(fd > 0) {
+            if(fd >= 0) {
                 close(fd);
                 fd = -1;
             }