commit c792c0f30dd0f67fc16fd0c0901eb719bf9b7d9b
author Pirama Arumuga Nainar <pirama@google.com> Tue Oct 09 16:58:14 2018 -0700
committer Pirama Arumuga Nainar <pirama@google.com> Mon Oct 15 15:52:44 2018 +0000
tree bbe51d14849ee36ed03f668b3b04f5d5b397aabc
parent 23bc16e71dcaf58c9ed2d3bcf6d92c2831ff6a0b

[hprof-conv] Check for NULL before closing descriptors

Else hprof-conf fails with a segfault.  Fixing this will help add a smoke
test for the Windows hprof-conv.

Test: hprof-conv.exe under wine
Change-Id: I0f19932ece26f0171e9c01c63536d71c28adb97f

tools/hprof-conv/HprofConv.c

diff --git a/tools/hprof-conv/HprofConv.c b/tools/hprof-conv/HprofConv.c
index a69440c..cc90176 100644
--- a/tools/hprof-conv/HprofConv.c
+++ b/tools/hprof-conv/HprofConv.c
@@ -764,9 +764,9 @@
     res = 2;
 
 finish:
-    if (in != stdin)
+    if (in != stdin && in != NULL)
         fclose(in);
-    if (out != stdout)
+    if (out != stdout && out != NULL)
         fclose(out);
     return res;
 }