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