Bug#754322: FTBFS with clang instead of gcc

[Alexander]

Package: libpam-chroot
Severity: minor
Tags: patch
User: pkg-llvm-t...@lists.alioth.debian.org
Usertags: clang-ftbfs

Hello,

Using the rebuild infrastructure, your package fails to build with clang 
(instead of gcc).

We detected this kinf of error:
http://clang.debian.net/status.php?version=3.4.2&key=TAUTOLOGICAL-COMPARE

Full build log is available here:
http://clang.debian.net/logs/2014-06-16/libpam-chroot_0.9-4.1_unstable_clang.log

Thanks,
Alexander

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-4-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

--- libpam-chroot-0.9/pam_chroot.c	2014-07-10 01:15:13.000000000 +0400
+++ libpam-chroot-0.9-my/pam_chroot.c	2014-07-10 01:14:56.711917765 +0400
@@ -187,7 +187,7 @@
 	char **uptr;
 
 	if(NULL == user || NULL == ngrps) { return -1; }
-	if(0 > *ngrps || 0 > gid) { return -1; }
+	if(0 > *ngrps || 0 == gid) { return -1; }
 
 	if(gcount < *ngrps) {
 		if(NULL != grps) { grps[gcount] = gid; }