From: Alejandro Colomar Date: Mon, 13 May 2024 12:46:47 +0000 (+0200) Subject: src/groupmems.c: Fix number of elements in allocation X-Git-Tag: 4.17.0-rc1~188 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=44ba094766e477ae08abd893091abbb58d1bf1e7;p=thirdparty%2Fshadow.git src/groupmems.c: Fix number of elements in allocation We are setting `sgrent.sg_adm[1] = NULL;`, so we need 2 elements. Fixes: 87b56b19fb72 ("* NEWS, src/groupmems.c, man/groupmems.8.xml: Added support for [...]") Signed-off-by: Alejandro Colomar --- diff --git a/src/groupmems.c b/src/groupmems.c index a369a6190..0d0882db2 100644 --- a/src/groupmems.c +++ b/src/groupmems.c @@ -127,7 +127,7 @@ static void add_user (const char *user, static struct sgrp sgrent; sgrent.sg_name = xstrdup (newgrp->gr_name); sgrent.sg_mem = dup_list (newgrp->gr_mem); - sgrent.sg_adm = XMALLOC(1, char *); + sgrent.sg_adm = XMALLOC(2, char *); #ifdef FIRST_MEMBER_IS_ADMIN if (sgrent.sg_mem[0]) { sgrent.sg_adm[0] = xstrdup (sgrent.sg_mem[0]); @@ -210,7 +210,7 @@ static void remove_user (const char *user, static struct sgrp sgrent; sgrent.sg_name = xstrdup (newgrp->gr_name); sgrent.sg_mem = dup_list (newgrp->gr_mem); - sgrent.sg_adm = XMALLOC(1, char *); + sgrent.sg_adm = XMALLOC(2, char *); #ifdef FIRST_MEMBER_IS_ADMIN if (sgrent.sg_mem[0]) { sgrent.sg_adm[0] = xstrdup (sgrent.sg_mem[0]);