lib.c: Use the size of the element when allocating an array.
authorMark Wooding <mdw@distorted.org.uk>
Thu, 13 May 2021 15:27:12 +0000 (16:27 +0100)
committerMark Wooding <mdw@distorted.org.uk>
Thu, 13 May 2021 15:27:12 +0000 (16:27 +0100)
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

diff --git a/lib.c b/lib.c
index 0d618f6..251d932 100644 (file)
--- 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);