Control: tags 919147 + pending Dear maintainer,
I've prepared an NMU for php-pear (versioned as 1:1.10.6+submodules+notgz-1.1) and uploaded it to DELAYED/10. Please feel free to tell me if I should delay it longer. Regards, Salvatore
diff -Nru php-pear-1.10.6+submodules+notgz/debian/changelog php-pear-1.10.6+submodules+notgz/debian/changelog --- php-pear-1.10.6+submodules+notgz/debian/changelog 2018-10-01 14:15:44.000000000 +0200 +++ php-pear-1.10.6+submodules+notgz/debian/changelog 2019-01-13 11:49:26.000000000 +0100 @@ -1,3 +1,11 @@ +php-pear (1:1.10.6+submodules+notgz-1.1) unstable; urgency=medium + + * Non-maintainer upload. + * Don't allow filenames to start with phar:// (CVE-2018-1000888) + (Closes: #919147) + + -- Salvatore Bonaccorso <car...@debian.org> Sun, 13 Jan 2019 11:49:26 +0100 + php-pear (1:1.10.6+submodules+notgz-1) unstable; urgency=medium * Update PEAR to 10.1.6 diff -Nru php-pear-1.10.6+submodules+notgz/debian/patches/0003-Don-t-allow-filenames-to-start-with-phar.patch php-pear-1.10.6+submodules+notgz/debian/patches/0003-Don-t-allow-filenames-to-start-with-phar.patch --- php-pear-1.10.6+submodules+notgz/debian/patches/0003-Don-t-allow-filenames-to-start-with-phar.patch 1970-01-01 01:00:00.000000000 +0100 +++ php-pear-1.10.6+submodules+notgz/debian/patches/0003-Don-t-allow-filenames-to-start-with-phar.patch 2019-01-13 11:49:26.000000000 +0100 @@ -0,0 +1,23 @@ +From: Michiel Rook <mr...@php.net> +Date: Thu, 20 Dec 2018 19:11:37 +0100 +Subject: Don't allow filenames to start with phar:// +Origin: https://github.com/pear/Archive_Tar/commit/59ace120ac5ceb5f0d36e40e48e1884de1badf76 +Bug-Debian-Security: https://security-tracker.debian.org/tracker/CVE-2018-1000888 +Bug-Debian: https://bugs.debian.org/919147 +Bug: https://pear.php.net/bugs/bug.php?id=23782 + +--- + +index 0bd1c6caae99..57a6d87f5b16 100644 +--- a/submodules/Archive_Tar/Archive/Tar.php ++++ b/submodules/Archive_Tar/Archive/Tar.php +@@ -1767,6 +1767,9 @@ class Archive_Tar extends PEAR + */ + private function _maliciousFilename($file) + { ++ if (strpos($file, 'phar://') === 0) { ++ return true; ++ } + if (strpos($file, '/../') !== false) { + return true; + } diff -Nru php-pear-1.10.6+submodules+notgz/debian/patches/series php-pear-1.10.6+submodules+notgz/debian/patches/series --- php-pear-1.10.6+submodules+notgz/debian/patches/series 2018-10-01 14:15:44.000000000 +0200 +++ php-pear-1.10.6+submodules+notgz/debian/patches/series 2019-01-13 11:49:26.000000000 +0100 @@ -1,2 +1,3 @@ 0001-pecl-command-needs-to-run-without-n-to-pickup-defaul.patch 0002-Fix-PECL-extensions-FTBFS-with-PHP-Fatal-error-Call-.patch +0003-Don-t-allow-filenames-to-start-with-phar.patch