[libcgi-application-plugin-authentication-perl] 03/05: drop cgi.pm-4.05.patch and fix-more-warnings.patch -- applied/fixed upstream

Damyan Ivanov dmn at moszumanska.debian.org
Sat Nov 4 09:49:24 UTC 2017


This is an automated email from the git hooks/post-receive script.

dmn pushed a commit to branch master
in repository libcgi-application-plugin-authentication-perl.

commit b2a571e5e90180c505bc378c8a7612dec0d5771d
Author: Damyan Ivanov <dmn at debian.org>
Date:   Sat Nov 4 09:44:53 2017 +0000

    drop cgi.pm-4.05.patch and fix-more-warnings.patch -- applied/fixed upstream
---
 debian/patches/cgi.pm-4.05.patch       | 21 ----------
 debian/patches/fix-more-warnings.patch | 72 ----------------------------------
 debian/patches/series                  |  2 -
 3 files changed, 95 deletions(-)

diff --git a/debian/patches/cgi.pm-4.05.patch b/debian/patches/cgi.pm-4.05.patch
deleted file mode 100644
index 2a318eb..0000000
--- a/debian/patches/cgi.pm-4.05.patch
+++ /dev/null
@@ -1,21 +0,0 @@
-Description: Adjust for CGI.pm >= 4.05
- CGI.pm now warns if parameters are imported in list context, as this is a potential security hole.
- Explicitly require scalar context.
-Origin: vendor
-Bug: https://rt.cpan.org/Ticket/Display.html?id=99460
-Forwarded: https://rt.cpan.org/Ticket/Display.html?id=99460
-Bug-Debian: https://bugs.debian.org/765051
-Author: gregor herrmann <gregoa at debian.org>
-Last-Update: 2014-10-13
-
---- a/lib/CGI/Application/Plugin/Authentication.pm
-+++ b/lib/CGI/Application/Plugin/Authentication.pm
-@@ -1205,7 +1205,7 @@
-     my $field_names = $config->{CREDENTIALS} || [qw(authen_username authen_password)];
- 
-     my $query = $self->_cgiapp->query;
--    my @credentials = map { $query->param($_) } @$field_names;
-+    my @credentials = map { scalar $query->param($_) } @$field_names;
-     if ($credentials[0]) {
-         # The user is trying to login
-         # make sure if they are already logged in, that we log them out first
diff --git a/debian/patches/fix-more-warnings.patch b/debian/patches/fix-more-warnings.patch
deleted file mode 100644
index d959c90..0000000
--- a/debian/patches/fix-more-warnings.patch
+++ /dev/null
@@ -1,72 +0,0 @@
-Description: [PATCH] Silent various warnings
- Some of them fail tests which sniffs for warnings. Some warning were
- introduced by CGI 4.05 or recent (5.20) perl.
-
- This is the same patch from the upstream tracker except the block that matched 
- the other CGI.pm patch was removed.
-
-Origin: other, https://rt.cpan.org/Ticket/Attachment/1495484/796570/CGI-Application-Plugin-Authentication-0.20-Silent-various-warnings.patch
-Bug: https://rt.cpan.org/Public/Bug/Display.html?id=99460
-Bug-Debian: https://bugs.debian.org/786447
-Author: Petr Písař <ppisar at redhat.com>
----
- lib/CGI/Application/Plugin/Authentication.pm                     | 1 +
- lib/CGI/Application/Plugin/Authentication/Driver/Filter/crypt.pm | 2 +-
- lib/CGI/Application/Plugin/Authentication/Driver/Filter/md5.pm   | 2 +-
- lib/CGI/Application/Plugin/Authentication/Driver/Filter/sha1.pm  | 2 +-
- 4 files changed, 5 insertions(+), 4 deletions(-)
-
-diff --git a/lib/CGI/Application/Plugin/Authentication.pm b/lib/CGI/Application/Plugin/Authentication.pm
-index 83230d2..2908925 100644
---- a/lib/CGI/Application/Plugin/Authentication.pm
-+++ b/lib/CGI/Application/Plugin/Authentication.pm
-@@ -767,6 +767,7 @@ sub is_protected_runmode {
-     my $self = shift;
-     my $runmode = shift;
- 
-+    require overload;
-     foreach my $runmode_test ($self->protected_runmodes) {
-         if (overload::StrVal($runmode_test) =~ /^Regexp=/) {
-             # We were passed a regular expression
-diff --git a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/crypt.pm b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/crypt.pm
-index 48dd28d..07377d4 100644
---- a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/crypt.pm
-+++ b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/crypt.pm
-@@ -15,7 +15,7 @@ sub check {
- 
- sub filter {
-     my $class = shift;
--    my $param = lc shift; # not used
-+    shift; # not used
-     my $plain = shift;
-     my $salt  = shift;
-     if (!$salt) {
-diff --git a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/md5.pm b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/md5.pm
-index 4b51ecf..56a0bea 100644
---- a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/md5.pm
-+++ b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/md5.pm
-@@ -25,7 +25,7 @@ sub check {
- 
- sub filter {
-     my $class = shift;
--    my $param = lc shift || 'hex';
-+    my $param = lc( shift || 'hex' );
-     my $plain = shift;
- 
-     Digest::MD5->require || die "Digest::MD5 is required to check MD5 passwords";
-diff --git a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/sha1.pm b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/sha1.pm
-index 610e9fa..b9f2f24 100644
---- a/lib/CGI/Application/Plugin/Authentication/Driver/Filter/sha1.pm
-+++ b/lib/CGI/Application/Plugin/Authentication/Driver/Filter/sha1.pm
-@@ -25,7 +25,7 @@ sub check {
- 
- sub filter {
-     my $class = shift;
--    my $param = lc shift || 'hex';
-+    my $param = lc( shift || 'hex' );
-     my $plain = shift;
- 
-     Digest::SHA->require || die "Digest::SHA is required to check SHA1 passwords";
--- 
-2.1.0
-
diff --git a/debian/patches/series b/debian/patches/series
index 66801a7..e69de29 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +0,0 @@
-cgi.pm-4.05.patch
-fix-more-warnings.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-perl/packages/libcgi-application-plugin-authentication-perl.git



More information about the Pkg-perl-cvs-commits mailing list