Hi David, thank you! I've done a minor edit below :
On Mon, Aug 22, 2016 at 11:38:02PM +0100, David CARLIER wrote: > @@ -6858,6 +6860,10 @@ cfg_parse_users(const char *file, int linenum, char > **args, int kwm) > Alert("parsing [%s:%d]: '%s' only supports > 'users' option.\n", > file, linenum, args[0]); > err_code |= ERR_ALERT | ERR_FATAL; > + if (ag->groupusers) > + free(ag->groupusers); removed the "if" above since it's already implemented in free() and we're trying to remove all of them for consistency. Applied now. thanks! Willy