From 44ba094766e477ae08abd893091abbb58d1bf1e7 Mon Sep 17 00:00:00 2001 From: Alejandro Colomar Date: Mon, 13 May 2024 14:46:47 +0200 Subject: [PATCH] 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 --- src/groupmems.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/groupmems.c b/src/groupmems.c index a369a619..0d0882db 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]);