From f50ecbe185f8866970ab62157b2e45bf613907d2 Mon Sep 17 00:00:00 2001 From: Mark Wooding Date: Thu, 13 May 2021 16:27:12 +0100 Subject: [PATCH] lib.c: Use the size of the element when allocating an array. Oops: a pure blunder. Fortunately, these are actually the same size on most current platforms, but that's not a good excuse. Spotted by Coverity. --- lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib.c b/lib.c index 0d618f6..251d932 100644 --- a/lib.c +++ b/lib.c @@ -932,7 +932,7 @@ static void set_config_section_parents(struct config *conf, * to have, so we can allocate the `parents' vector and fill it in. */ sect->nparents = av.n/2; - sect->parents = xmalloc(sect->nparents*sizeof(sect->parents)); + sect->parents = xmalloc(sect->nparents*sizeof(*sect->parents)); for (i = 0; i < av.n; i += 2) { n = av.v[i + 1] - av.v[i]; parent = config_find_section_n(conf, 0, av.v[i], n); -- 2.11.0