shithub: choc

Info  •  Files  •  Log  •  Branches

explicitly direct output to stdout if '-' is given as a file name

There is no mention in the POSIX documentation that fprintf() will
print to stdout if the NULL pointer is passed as the stream.
And, indeed, the current implementation triggers a segmentation fault
if invoked as '-statdump -'.

--- a/src/doom/statdump.c

+++ b/src/doom/statdump.c

@@ -339,7 +339,7 @@

         }

         else

         {

-            dumpfile = NULL;

+            dumpfile = stdout;

         }

         for (i = 0; i < num_captured_stats; ++i)

@@ -347,7 +347,7 @@

             PrintStats(dumpfile, &captured_stats[i]);

         }

-        if (dumpfile != NULL)

+        if (dumpfile != stdout)

         {

             fclose(dumpfile);

         }