Here's a NMU for this and #754322, using Jason Duerstock's patch.
Will upload to DELAYED/7; preferred way to cancel is dput with a better
version, barring that dcut or shouting at me should also work.


Meow!
-- 
⢀⣴⠾⠻⢶⣦⠀ 
⣾⠁⢰⠒⠀⣿⡁ 
⢿⡄⠘⠷⠚⠋⠀ ... what's the frequency of that 5V DC?
⠈⠳⣄⠀⠀⠀⠀
diff -u libpam-chroot-0.9/debian/changelog libpam-chroot-0.9/debian/changelog
--- libpam-chroot-0.9/debian/changelog
+++ libpam-chroot-0.9/debian/changelog
@@ -1,3 +1,11 @@
+libpam-chroot (0.9-4.3) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Fix FTBFS with GCC-7 and Clang. Closes: 853502, #754322; thanks Jason
+    Duerstock, Juhani Numminen.
+
+ -- Adam Borowski <kilob...@angband.pl>  Wed, 11 Apr 2018 02:18:46 +0200
+
 libpam-chroot (0.9-4.2) unstable; urgency=medium
 
   * Non-maintainer upload.
diff -u libpam-chroot-0.9/pam_chroot.c libpam-chroot-0.9/pam_chroot.c
--- libpam-chroot-0.9/pam_chroot.c
+++ libpam-chroot-0.9/pam_chroot.c
@@ -187,7 +187,7 @@
        char **uptr;
 
        if(NULL == user || NULL == ngrps) { return -1; }
-       if(0 > *ngrps || 0 > gid) { return -1; }
+       if(0 > *ngrps) { return -1; }
 
        if(gcount < *ngrps) {
                if(NULL != grps) { grps[gcount] = gid; }
@@ -870,7 +870,7 @@
                        break;
 
                default:
-                       if(opts.flags * _PAM_OPTS_DEBUG) {
+                       if(opts.flags & _PAM_OPTS_DEBUG) {
                                _pam_log(LOG_NOTICE, "%s: returning failure", 
opts.module);
                        }
                        err = PAM_SESSION_ERR;

Reply via email to