Comment 1 for bug 1537797

From the Debian report:

The system headers define:
extern struct passwd *getpwnam (const char *__name) __attribute__ ((__nonnull__ (1)));
extern int __attribute__((__nonnull__ (1))) pam_end(pam_handle_t *pamh, int pam_status);

So the compiler considers that 'name' can never be NULL and my check is therefore an error.

In this case it's probably safe to drop these checks. because these are just mock functions.