Delivery: fix memory leak
authorJeremy Harris <jgh146exb@wizmail.org>
Sat, 24 Sep 2016 16:11:19 +0000 (17:11 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Sat, 24 Sep 2016 16:11:19 +0000 (17:11 +0100)
src/src/deliver.c

index 357c607..eb6c705 100644 (file)
@@ -5555,7 +5555,11 @@ Otherwise it might be needed again. */
        "journal file\n", big_buffer);
       }
     rewind(jread);
-    journal_fd = fileno(jread);
+    if ((journal_fd = dup(fileno(jread))) < 0)
+      journal_fd = fileno(jread);
+    else
+      (void) fclose(jread);    /* Try to not leak the FILE resource */
+
     /* Panic-dies on error */
     (void)spool_write_header(message_id, SW_DELIVERING, NULL);
     }