From f529c9a3936003f01cd4a0052fc5263734a0c92d Mon Sep 17 00:00:00 2001
From: baldurk <baldurk@baldurk.org>
Date: Mon, 4 Sep 2017 11:14:33 +0100
Subject: [PATCH] Don't use logfile FD if it's -1

---
 renderdoc/os/posix/posix_stringio.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/renderdoc/os/posix/posix_stringio.cpp b/renderdoc/os/posix/posix_stringio.cpp
index 0c1847da1..6bfbeb7b0 100644
--- a/renderdoc/os/posix/posix_stringio.cpp
+++ b/renderdoc/os/posix/posix_stringio.cpp
@@ -454,7 +454,8 @@ void ReleaseFDAfterFork()
 {
   // we do NOT release the shared lock here, since the file descriptor is shared so we'd be
   // releasing the parent process's lock. Just close our file descriptor
-  close(logfileFD);
+  if(logfileFD >= 0)
+    close(logfileFD);
 }
 
 bool logfile_open(const char *filename)
@@ -474,13 +475,13 @@ bool logfile_open(const char *filename)
 
 void logfile_append(const char *msg, size_t length)
 {
-  if(logfileFD)
+  if(logfileFD >= 0)
     write(logfileFD, msg, (unsigned int)length);
 }
 
 void logfile_close(const char *filename)
 {
-  if(logfileFD)
+  if(logfileFD >= 0)
   {
     // release our shared lock
     int err = flock(logfileFD, LOCK_UN | LOCK_NB);