]> granicus.if.org Git - graphviz/commitdiff
pathplan Pobsopen: use a 'size_t' when calling 'malloc'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 30 Jul 2022 16:45:16 +0000 (09:45 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 5 Aug 2022 13:52:10 +0000 (06:52 -0700)
Squashes a -Wsign-conversion warning and makes explicit a previously implicit
assumption.

lib/pathplan/cvt.c

index b1654ec3bcf860923c2d00f1532367749a9f0801..aa1e7b52b0d2a3f6f6f3639aa0ee95ed84e0f779 100644 (file)
@@ -52,7 +52,8 @@ vconfig_t *Pobsopen(Ppoly_t ** obs, int n_obs)
        return NULL;
     }
     rv->P = malloc(n * sizeof(Ppoint_t));
-    rv->start = malloc((n_obs + 1) * sizeof(int));
+    assert(n_obs >= 0);
+    rv->start = malloc(((size_t)n_obs + 1) * sizeof(int));
     rv->next = malloc(n * sizeof(int));
     rv->prev = malloc(n * sizeof(int));
     rv->N = (int)n;