From: PatR <rankin@nethack.org>
Date: Tue, 31 May 2022 17:36:12 +0000 (-0700)
Subject: another re_alloc() bit
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=42981361d75cdee5f6588d13ab3dab2366b3a184;p=nethack

another re_alloc() bit

realloc(NULL, size) is legitimate usage and nhrealloc() shouldn't
log a "<  0x00000000 __FILE__ __LINE__" entry for it.  heaputil
would complain about freeing Null.
---

diff --git a/src/alloc.c b/src/alloc.c
index 244dc2cb8..8113a9389 100644
--- a/src/alloc.c
+++ b/src/alloc.c
@@ -164,9 +164,10 @@ nhrealloc(
                         * allocation rather than make a new one */
 
         if (newptr != oldptr) {
-            /* realloc() freed oldptr */
-            (void) fprintf(heaplog, "%c%5s %s %4d %s\n", '<', "",
-                           fmt_ptr((genericptr_t) oldptr), line, file);
+            /* if oldptr wasn't Null, realloc() freed it */
+            if (oldptr)
+                (void) fprintf(heaplog, "%c%5s %s %4d %s\n", '<', "",
+                               fmt_ptr((genericptr_t) oldptr), line, file);
             op = '>'; /* new allocation rather than size-change of old one */
         }
         (void) fprintf(heaplog, "%c%5u %s %4d %s\n", op, newlth,