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

Reply via email to