[arch-commits] Commit in lib32-pam (6 files)
Maxime Gauduin
alucryd at archlinux.org
Sun Oct 11 15:23:01 UTC 2015
Date: Sunday, October 11, 2015 @ 17:23:01
Author: alucryd
Revision: 143656
archrelease: copy trunk to multilib-x86_64
Added:
lib32-pam/repos/
lib32-pam/repos/multilib-x86_64/
lib32-pam/repos/multilib-x86_64/PKGBUILD
(from rev 143655, lib32-pam/trunk/PKGBUILD)
lib32-pam/repos/multilib-x86_64/pam-1.1.8-cve-2013-7041.patch
(from rev 143655, lib32-pam/trunk/pam-1.1.8-cve-2013-7041.patch)
lib32-pam/repos/multilib-x86_64/pam-1.1.8-cve-2014-2583.patch
(from rev 143655, lib32-pam/trunk/pam-1.1.8-cve-2014-2583.patch)
lib32-pam/repos/multilib-x86_64/pam_unix2-glibc216.patch
(from rev 143655, lib32-pam/trunk/pam_unix2-glibc216.patch)
-------------------------------+
PKGBUILD | 75 ++++++++++++++++++++++++++++++++++++++++
pam-1.1.8-cve-2013-7041.patch | 52 +++++++++++++++++++++++++++
pam-1.1.8-cve-2014-2583.patch | 56 +++++++++++++++++++++++++++++
pam_unix2-glibc216.patch | 20 ++++++++++
4 files changed, 203 insertions(+)
Copied: lib32-pam/repos/multilib-x86_64/PKGBUILD (from rev 143655, lib32-pam/trunk/PKGBUILD)
===================================================================
--- repos/multilib-x86_64/PKGBUILD (rev 0)
+++ repos/multilib-x86_64/PKGBUILD 2015-10-11 15:23:01 UTC (rev 143656)
@@ -0,0 +1,75 @@
+# $Id$
+# Maintainer: Maxime Gauduin <alucryd at archlinux.org>
+# Contributor: jtts <jussaar at mbnet.fi>
+# Contributor: josephgbr <rafael.f.f1 at gmail.com>
+# Contributor: Janax <janax99 at yahoo.com>
+# Contributor: Tobias Powalowski <tpowa at archlinux.org>
+# Contributor: judd <jvinet at zeroflux.org>
+
+pkgname=lib32-pam
+pkgver=1.2.1
+pkgrel=1
+pkgdesc='PAM (Pluggable Authentication Modules)'
+arch=('x86_64')
+url='http://linux-pam.org'
+license=('GPL2')
+depends=('lib32-cracklib' 'lib32-libtirpc' 'pam')
+makedepends=('gcc-multilib' 'lib32-flex')
+source=("http://linux-pam.org/library/Linux-PAM-${pkgver}.tar.bz2"
+ 'https://sources.archlinux.org/other/pam_unix2/pam_unix2-2.9.1.tar.bz2'
+ 'pam_unix2-glibc216.patch')
+sha256sums=('342b1211c0d3b203a7df2540a5b03a428a087bd8a48c17e49ae268f992b334d9'
+ '3315747699fece4e1cc5771885d243b3e017c4c4ca1326e86228d590a840e955'
+ '6644c5cff46878c65bdc77977becbeda392675702264bfcc7c610a45a9982574')
+options=('!emptydirs')
+
+prepare () {
+ cd pam_unix2-2.9.1
+
+ patch -Np1 -i ../pam_unix2-glibc216.patch
+}
+
+build() {
+ cd Linux-PAM-${pkgver}
+
+ export CC='gcc -m32'
+ export CXX='g++ -m32'
+ export PKG_CONFIG_PATH='/usr/lib32/pkgconfig'
+
+ ./configure \
+ --prefix='/usr' \
+ --libdir='/usr/lib32' \
+ --sbindir='/usr/bin' \
+ --disable-db
+ make
+
+ cd ../pam_unix2-2.9.1
+
+ export CFLAGS="$CFLAGS -I"${srcdir}"/Linux-PAM-${pkgver}/libpam/include/"
+ export LDFLAGS="$LDFLAGS -L"${srcdir}"/Linux-PAM-${pkgver}/libpam/.libs/"
+
+ ./configure --prefix='/usr' \
+ --libdir='/usr/lib32' \
+ --sbindir='/usr/bin'
+ make
+}
+
+package() {
+ cd Linux-PAM-${pkgver}
+
+ make DESTDIR="${pkgdir}" SCONFIGDIR='/etc/security' install
+ rm -rf "${pkgdir}"/{etc,usr/{include,share,bin}}
+
+ cd ../pam_unix2-2.9.1
+
+ install -m 644 src/pam_unix2.so "${pkgdir}"/usr/lib32/security/
+
+ cd "${pkgdir}"/usr/lib32/security
+
+ ln -s pam_unix.so pam_unix_acct.so
+ ln -s pam_unix.so pam_unix_auth.so
+ ln -s pam_unix.so pam_unix_passwd.so
+ ln -s pam_unix.so pam_unix_session.so
+}
+
+# vim: ts=2 sw=2 et:
Copied: lib32-pam/repos/multilib-x86_64/pam-1.1.8-cve-2013-7041.patch (from rev 143655, lib32-pam/trunk/pam-1.1.8-cve-2013-7041.patch)
===================================================================
--- repos/multilib-x86_64/pam-1.1.8-cve-2013-7041.patch (rev 0)
+++ repos/multilib-x86_64/pam-1.1.8-cve-2013-7041.patch 2015-10-11 15:23:01 UTC (rev 143656)
@@ -0,0 +1,52 @@
+From 57a1e2b274d0a6376d92ada9926e5c5741e7da20 Mon Sep 17 00:00:00 2001
+From: "Dmitry V. Levin" <ldv at altlinux.org>
+Date: Fri, 24 Jan 2014 22:18:32 +0000
+Subject: [PATCH] pam_userdb: fix password hash comparison
+
+Starting with commit Linux-PAM-0-77-28-g0b3e583 that introduced hashed
+passwords support in pam_userdb, hashes are compared case-insensitively.
+This bug leads to accepting hashes for completely different passwords in
+addition to those that should be accepted.
+
+Additionally, commit Linux-PAM-1_1_6-13-ge2a8187 that added support for
+modern password hashes with different lengths and settings, did not
+update the hash comparison accordingly, which leads to accepting
+computed hashes longer than stored hashes when the latter is a prefix
+of the former.
+
+* modules/pam_userdb/pam_userdb.c (user_lookup): Reject the computed
+hash whose length differs from the stored hash length.
+Compare computed and stored hashes case-sensitively.
+Fixes CVE-2013-7041.
+
+Bug-Debian: http://bugs.debian.org/731368
+---
+ modules/pam_userdb/pam_userdb.c | 9 ++++++---
+ 1 file changed, 6 insertions(+), 3 deletions(-)
+
+diff --git a/modules/pam_userdb/pam_userdb.c b/modules/pam_userdb/pam_userdb.c
+index de8b5b1..ff040e6 100644
+--- a/modules/pam_userdb/pam_userdb.c
++++ b/modules/pam_userdb/pam_userdb.c
+@@ -222,12 +222,15 @@ user_lookup (pam_handle_t *pamh, const char *database, const char *cryptmode,
+ } else {
+ cryptpw = crypt (pass, data.dptr);
+
+- if (cryptpw) {
+- compare = strncasecmp (data.dptr, cryptpw, data.dsize);
++ if (cryptpw && strlen(cryptpw) == (size_t)data.dsize) {
++ compare = memcmp(data.dptr, cryptpw, data.dsize);
+ } else {
+ compare = -2;
+ if (ctrl & PAM_DEBUG_ARG) {
+- pam_syslog(pamh, LOG_INFO, "crypt() returned NULL");
++ if (cryptpw)
++ pam_syslog(pamh, LOG_INFO, "lengths of computed and stored hashes differ");
++ else
++ pam_syslog(pamh, LOG_INFO, "crypt() returned NULL");
+ }
+ };
+
+--
+1.8.3.1
+
Copied: lib32-pam/repos/multilib-x86_64/pam-1.1.8-cve-2014-2583.patch (from rev 143655, lib32-pam/trunk/pam-1.1.8-cve-2014-2583.patch)
===================================================================
--- repos/multilib-x86_64/pam-1.1.8-cve-2014-2583.patch (rev 0)
+++ repos/multilib-x86_64/pam-1.1.8-cve-2014-2583.patch 2015-10-11 15:23:01 UTC (rev 143656)
@@ -0,0 +1,56 @@
+From 9dcead87e6d7f66d34e7a56d11a30daca367dffb Mon Sep 17 00:00:00 2001
+From: "Dmitry V. Levin" <ldv at altlinux.org>
+Date: Wed, 26 Mar 2014 22:17:23 +0000
+Subject: [PATCH] pam_timestamp: fix potential directory traversal issue
+ (ticket #27)
+
+pam_timestamp uses values of PAM_RUSER and PAM_TTY as components of
+the timestamp pathname it creates, so extra care should be taken to
+avoid potential directory traversal issues.
+
+* modules/pam_timestamp/pam_timestamp.c (check_tty): Treat
+"." and ".." tty values as invalid.
+(get_ruser): Treat "." and ".." ruser values, as well as any ruser
+value containing '/', as invalid.
+
+Fixes CVE-2014-2583.
+
+Reported-by: Sebastian Krahmer <krahmer at suse.de>
+---
+ modules/pam_timestamp/pam_timestamp.c | 13 ++++++++++++-
+ 1 file changed, 12 insertions(+), 1 deletion(-)
+
+diff --git a/modules/pam_timestamp/pam_timestamp.c b/modules/pam_timestamp/pam_timestamp.c
+index 5193733..b3f08b1 100644
+--- a/modules/pam_timestamp/pam_timestamp.c
++++ b/modules/pam_timestamp/pam_timestamp.c
+@@ -158,7 +158,7 @@ check_tty(const char *tty)
+ tty = strrchr(tty, '/') + 1;
+ }
+ /* Make sure the tty wasn't actually a directory (no basename). */
+- if (strlen(tty) == 0) {
++ if (!strlen(tty) || !strcmp(tty, ".") || !strcmp(tty, "..")) {
+ return NULL;
+ }
+ return tty;
+@@ -243,6 +243,17 @@ get_ruser(pam_handle_t *pamh, char *ruserbuf, size_t ruserbuflen)
+ if (pwd != NULL) {
+ ruser = pwd->pw_name;
+ }
++ } else {
++ /*
++ * This ruser is used by format_timestamp_name as a component
++ * of constructed timestamp pathname, so ".", "..", and '/'
++ * are disallowed to avoid potential path traversal issues.
++ */
++ if (!strcmp(ruser, ".") ||
++ !strcmp(ruser, "..") ||
++ strchr(ruser, '/')) {
++ ruser = NULL;
++ }
+ }
+ if (ruser == NULL || strlen(ruser) >= ruserbuflen) {
+ *ruserbuf = '\0';
+--
+1.8.3.1
+
Copied: lib32-pam/repos/multilib-x86_64/pam_unix2-glibc216.patch (from rev 143655, lib32-pam/trunk/pam_unix2-glibc216.patch)
===================================================================
--- repos/multilib-x86_64/pam_unix2-glibc216.patch (rev 0)
+++ repos/multilib-x86_64/pam_unix2-glibc216.patch 2015-10-11 15:23:01 UTC (rev 143656)
@@ -0,0 +1,20 @@
+Index: pam_unix2-2.9.1/src/read-files.c
+===================================================================
+--- pam_unix2-2.9.1.orig/src/read-files.c
++++ pam_unix2-2.9.1/src/read-files.c
+@@ -30,8 +30,14 @@
+ #include <errno.h>
+ #include <fcntl.h>
+ #include <nss.h>
+-#include <bits/libc-lock.h>
++#include <pthread.h>
+ #define __libc_lock_t pthread_mutex_t
++#define __libc_lock_define_initialized(CLASS,NAME) \
++ CLASS __libc_lock_t NAME = PTHREAD_MUTEX_INITIALIZER;
++#define __libc_lock_lock(NAME) \
++ pthread_mutex_lock, (&(NAME))
++#define __libc_lock_unlock(NAME) \
++ pthread_mutex_unlock, (&(NAME))
+
+ #include "read-files.h"
+
More information about the arch-commits
mailing list