]> granicus.if.org Git - graphviz/commitdiff
refactor a GNEW call that can tolerate failure into a direct malloc
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 15 Jun 2020 23:16:36 +0000 (16:16 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 22 Jun 2020 23:39:21 +0000 (16:39 -0700)
We are about to make the GNEW wrapper exit on allocation failure, so this
preserves the existing semantics of allocation failure being non-fatal to this
call. This is the only GNEW call I can see that can tolerate allocation failure.

lib/pack/ccomps.c

index 9f8d27c9676ed40818e213922c858ac6d1a27ccc..196239ed42e940d420e8c7685dbed97c9ebd47a0 100644 (file)
@@ -14,6 +14,7 @@
 
 #include <ctype.h>
 #include <setjmp.h>
+#include <stdlib.h>
 #include "render.h"
 #include "pack.h"
 
@@ -74,7 +75,7 @@ static void push(stk_t* sp, Agnode_t * np)
 {
     if (sp->curp == sp->curblk->endp) {
        if (sp->curblk->next == NULL) {
-           blk_t *bp = GNEW(blk_t);
+           blk_t *bp = malloc(sizeof(blk_t));
            if (bp == 0) {
                agerr(AGERR, "gc: Out of memory\n");
                longjmp(jbuf, 1);