On Tue, Jun 27, 2017 at 01:10:19PM +0200, Tom de Vries wrote:
> +/* Secure getenv() which returns NULL if running as SUID/SGID.  */
> +#ifndef HAVE_SECURE_GETENV
> +#if defined (HAVE_UNISTD_H) && defined(HAVE_GETUID) && defined(HAVE_GETEUID) 
> \
> +  && defined(HAVE_GETGID) && defined(HAVE_GETEGID)

Can you please us consistent formatting (i.e. space between defined and (
everywhere in the patch)?  The above line will be too long, so you'll need
to split it into 3 lines.

Ok with that change, thanks.

        Jakub

Reply via email to