From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 30 Jul 2022 16:45:16 +0000 (-0700)
Subject: pathplan Pobsopen: use a 'size_t' when calling 'malloc'
X-Git-Tag: 5.0.1~21^2~5
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=61475dada9375fdea04a72feda88d1aee42b82a5;p=graphviz

pathplan Pobsopen: use a 'size_t' when calling 'malloc'

Squashes a -Wsign-conversion warning and makes explicit a previously implicit
assumption.
---

diff --git a/lib/pathplan/cvt.c b/lib/pathplan/cvt.c
index b1654ec3b..aa1e7b52b 100644
--- a/lib/pathplan/cvt.c
+++ b/lib/pathplan/cvt.c
@@ -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;