xmkstemp: UB, close handles correctly

* free tempfile

1 files changed, 6 insertions, 10 deletions

diff --git a/src/lib/core/utils.c b/src/lib/core/utils.c
index 0ba5170..35310b4 100644
--- a/src/lib/core/utils.c
+++ b/src/lib/core/utils.c
@@ -525,23 +525,19 @@ char *xmkstemp(FILE **fp, const char *mode) {
     }
     *fp = fdopen(fd, mode);
     if (!*fp) {
-        // unable to open, die
-        if (fd > 0)
-            close(fd);
-        *fp = NULL;
+        close(fd);
         return NULL;
     }
 
     char *path = strdup(t_name);
     if (!path) {
         // strdup failed, die
-        if (*fp) {
-            // close the file handle
-            fclose(*fp);
-            *fp = NULL;
-        }
-        // fall through. path is NULL.
+        // close the file handle
+        fclose(*fp);
+        *fp = NULL;
+        return NULL;
     }
+
     return path;
 }