pool/perl-DBI
SHA256
4
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Accepting request 1236807 from devel:languages:perl

Browse Source 
- Remove perl-DBI-CVE-2014-10402.patch (fixed upstream)
- updated to 1.645.0 (1.645)
   see /usr/share/doc/packages/perl-DBI/Changes (forwarded request 1236685 from tinita)

OBS-URL: https://build.opensuse.org/request/show/1236807
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/perl-DBI?expand=0&rev=53
This commit is contained in:
Ana Guerrero 2025-01-12 10:08:35 +00:00 committed by Git OBS Bridge
commit 5bb4caa765
6 changed files with 111 additions and 276 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

BIN
DBI-1.643.tar.gz (Stored with Git LFS)
View File

Binary file not shown.

3
DBI-1.645.tgz Normal file
View File

@ -0,0 +1,3 @@
version https://git-lfs.github.com/spec/v1
oid sha256:e38b7a5efee129decda12383cf894963da971ffac303f54cc1b93e40e3cf9921
size 728223

2
cpanspec.yml
View File

@ -4,7 +4,7 @@
sources:
- perl-DBI.rpmlintrc
patches:
perl-DBI-CVE-2014-10402.patch: -p1
# perl-DBI-CVE-2014-10402.patch: -p1
#preamble: |-
# BuildRequires: gcc-c++
#post_prep: |-

257
perl-DBI-CVE-2014-10402.patch
View File

@ -1,257 +0,0 @@
From 32398bff24f054f4e9b48b97ecb70ce72267296c Mon Sep 17 00:00:00 2001
From: Jens Rehsack <sno@netbsd.org>
Date: Tue, 6 Oct 2020 06:50:37 +0200
Subject: [PATCH 1/3] DBD/File,DBI/DBD/SqlEngine: bump copyright year
Bump copyright years for both since there has been done work in meantime ...
including intended f_dir= fix for CVE-2014-10401
Signed-off-by: Jens Rehsack <sno@netbsd.org>
---
lib/DBD/File.pm | 4 ++--
lib/DBI/DBD/SqlEngine.pm | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/lib/DBD/File.pm b/lib/DBD/File.pm
index dd4312b..fb14e9a 100644
--- a/lib/DBD/File.pm
+++ b/lib/DBD/File.pm
@@ -9,7 +9,7 @@
#
# The original author is Jochen Wiedmann.
#
-# Copyright (C) 2009-2013 by H.Merijn Brand & Jens Rehsack
+# Copyright (C) 2009-2020 by H.Merijn Brand & Jens Rehsack
# Copyright (C) 2004 by Jeff Zucker
# Copyright (C) 1998 by Jochen Wiedmann
#
@@ -1430,7 +1430,7 @@ The original author is Jochen Wiedmann.
=head1 COPYRIGHT AND LICENSE
- Copyright (C) 2009-2013 by H.Merijn Brand & Jens Rehsack
+ Copyright (C) 2009-2020 by H.Merijn Brand & Jens Rehsack
Copyright (C) 2004-2009 by Jeff Zucker
Copyright (C) 1998-2004 by Jochen Wiedmann
diff --git a/lib/DBI/DBD/SqlEngine.pm b/lib/DBI/DBD/SqlEngine.pm
index fb755ab..8e933f7 100644
--- a/lib/DBI/DBD/SqlEngine.pm
+++ b/lib/DBI/DBD/SqlEngine.pm
@@ -9,7 +9,7 @@
#
# The original author is Jochen Wiedmann.
#
-# Copyright (C) 2009-2013 by H.Merijn Brand & Jens Rehsack
+# Copyright (C) 2009-2020 by H.Merijn Brand & Jens Rehsack
# Copyright (C) 2004 by Jeff Zucker
# Copyright (C) 1998 by Jochen Wiedmann
#
@@ -2216,7 +2216,7 @@ The original authors are Jochen Wiedmann and Jeff Zucker.
=head1 COPYRIGHT AND LICENSE
- Copyright (C) 2009-2013 by H.Merijn Brand & Jens Rehsack
+ Copyright (C) 2009-2020 by H.Merijn Brand & Jens Rehsack
Copyright (C) 2004-2009 by Jeff Zucker
Copyright (C) 1998-2004 by Jochen Wiedmann
From 27b10b5c3aacabc091046beaba478e671bb6111c Mon Sep 17 00:00:00 2001
From: Jens Rehsack <sno@netbsd.org>
Date: Tue, 6 Oct 2020 08:23:55 +0200
Subject: [PATCH 2/3] t/51dbm_file.t: add test from RT#99508
Add test with f_dir="something-not-existing" as reported in RT#99508
to verify when it's fixed for real.
Signed-off-by: Jens Rehsack <sno@netbsd.org>
---
t/51dbm_file.t | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/t/51dbm_file.t b/t/51dbm_file.t
index d9824cf..686a3d2 100644
--- a/t/51dbm_file.t
+++ b/t/51dbm_file.t
@@ -15,6 +15,27 @@ use DBI;
do "./t/lib.pl";
+{
+ # test issue reported in RT#99508
+ my @msg;
+ eval {
+ local $SIG{__DIE__} = sub { push @msg, @_ };
+ my $dbh = DBI->connect ("dbi:DBM:f_dir=./hopefully-doesnt-existst;sql_identifier_case=1;RaiseError=1");
+ };
+ like ("@msg", qr{.*hopefully-doesnt-existst.*}, "Cannot open from non-existing directory with attributes in DSN");
+
+ @msg = ();
+ eval {
+ local $SIG{__DIE__} = sub { push @msg, @_ };
+ my $dbh = DBI->connect ("dbi:DBM:", , undef, undef, {
+ f_dir => "./hopefully-doesnt-existst",
+ sql_identifier_case => 1,
+ RaiseError => 1,
+ });
+ };
+ like ("@msg", qr{.*hopefully-doesnt-existst}, "Cannot open from non-existing directory with attributes in HASH");
+}
+
my $dir = test_dir();
my $dbh = DBI->connect( 'dbi:DBM:', undef, undef, {
@@ -23,6 +44,8 @@ my $dbh = DBI->connect( 'dbi:DBM:', undef, undef, {
}
);
+ok( $dbh, "Connect with driver attributes in hash" );
+
ok( $dbh->do(q/drop table if exists FRED/), 'drop table' );
my $dirfext = $^O eq 'VMS' ? '.sdbm_dir' : '.dir';
From 19d0fb169eed475e1c053e99036b8668625cfa94 Mon Sep 17 00:00:00 2001
From: Jens Rehsack <sno@netbsd.org>
Date: Tue, 6 Oct 2020 10:22:17 +0200
Subject: [PATCH 3/3] lib/DBD/File.pm: fix CVE-2014-10401
Dig into the root cause of RT#99508 - which resulted in CVE-2014-10401 - and
figure out that DBI->parse_dsn is the wrong helper to parse our attributes in
DSN, since in DBD::dr::connect only the "dbname" remains from DSN which causes
parse_dsn to bailout.
Parsing on our own similar to parse_dsn shows the way out.
Signed-off-by: Jens Rehsack <sno@netbsd.org>
---
lib/DBD/File.pm | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/lib/DBD/File.pm b/lib/DBD/File.pm
index fb14e9a..f55076f 100644
--- a/lib/DBD/File.pm
+++ b/lib/DBD/File.pm
@@ -109,7 +109,11 @@ sub connect
# We do not (yet) care about conflicting attributes here
# my $dbh = DBI->connect ("dbi:CSV:f_dir=test", undef, undef, { f_dir => "text" });
# will test here that both test and text should exist
- if (my $attr_hash = (DBI->parse_dsn ($dbname))[3]) {
+ #
+ # Parsing on our own similar to parse_dsn to find attributes in 'dbname' parameter.
+ if ($dbname) {
+ my @attrs = split /;/ => $dbname;
+ my $attr_hash = { map { split /\s*=>?\s*|\s*,\s*/, $_} @attrs };
if (defined $attr_hash->{f_dir} && ! -d $attr_hash->{f_dir}) {
my $msg = "No such directory '$attr_hash->{f_dir}";
$drh->set_err (2, $msg);
@@ -120,7 +124,6 @@ sub connect
if ($attr and defined $attr->{f_dir} && ! -d $attr->{f_dir}) {
my $msg = "No such directory '$attr->{f_dir}";
$drh->set_err (2, $msg);
- $attr->{RaiseError} and croak $msg;
return;
}
From c71b64a678bcd708c7b75ee2bef5a360c836444c Mon Sep 17 00:00:00 2001
From: "H.Merijn Brand - Tux" <linux@tux.freedom.nl>
Date: Wed, 28 Oct 2020 15:57:17 +0100
Subject: [PATCH] Document the new behavior for f_dir
These changes also warrant a version increase
---
lib/DBD/File.pm | 5 +++++
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/lib/DBD/File.pm b/lib/DBD/File.pm
index baffefa..afbff26 100644
--- a/lib/DBD/File.pm
+++ b/lib/DBD/File.pm
@@ -1062,6 +1062,11 @@ directory) when the dbh attribute is set.
f_dir => "/data/foo/csv",
+If C<f_dir> is set to a non-existing location, the connection will fail.
+See CVE-2014-10401 for reasoning. Because of this, folders to use cannot
+be created after the connection, but must exist before the connection is
+initiated.
+
See L<KNOWN BUGS AND LIMITATIONS>.
=head4 f_dir_search
From 89f0d4cd38b83f0ee426a5fdf7d1ad5ea371c883 Mon Sep 17 00:00:00 2001
From: "H.Merijn Brand - Tux" <linux@tux.freedom.nl>
Date: Wed, 28 Oct 2020 15:03:48 +0100
Subject: [PATCH] Fix for empty attributes in DSN
dbm_type=SDBM_File;dbm_mldbm=;f_lockfile=.lck'
^
would result in
Odd number of elements in anonymous hash
---
lib/DBD/File.pm | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/DBD/File.pm b/lib/DBD/File.pm
index f55076f..baffefa 100644
--- a/lib/DBD/File.pm
+++ b/lib/DBD/File.pm
@@ -112,8 +112,9 @@ sub connect
#
# Parsing on our own similar to parse_dsn to find attributes in 'dbname' parameter.
if ($dbname) {
- my @attrs = split /;/ => $dbname;
- my $attr_hash = { map { split /\s*=>?\s*|\s*,\s*/, $_} @attrs };
+ my $attr_hash = {
+ map { (m/^\s* (\S+) \s*(?: =>? | , )\s* (\S*) \s*$/x) }
+ split m/;/ => $dbname };
if (defined $attr_hash->{f_dir} && ! -d $attr_hash->{f_dir}) {
my $msg = "No such directory '$attr_hash->{f_dir}";
$drh->set_err (2, $msg);
From 2eda0ec996d0a9357885acd442c72ac206adb7b3 Mon Sep 17 00:00:00 2001
From: "H.Merijn Brand - Tux" <linux@tux.freedom.nl>
Date: Wed, 28 Oct 2020 15:09:01 +0100
Subject: [PATCH] Catch warning
---
t/51dbm_file.t | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/t/51dbm_file.t b/t/51dbm_file.t
index 686a3d2..0ae910c 100644
--- a/t/51dbm_file.t
+++ b/t/51dbm_file.t
@@ -18,21 +18,25 @@ do "./t/lib.pl";
{
# test issue reported in RT#99508
my @msg;
- eval {
- local $SIG{__DIE__} = sub { push @msg, @_ };
- my $dbh = DBI->connect ("dbi:DBM:f_dir=./hopefully-doesnt-existst;sql_identifier_case=1;RaiseError=1");
+ my $dbh = eval {
+ local $SIG{__WARN__} = sub { push @msg, @_ };
+ local $SIG{__DIE__} = sub { push @msg, @_ };
+ DBI->connect ("dbi:DBM:f_dir=./hopefully-doesnt-existst;sql_identifier_case=1;RaiseError=1");
};
+ is ($dbh, undef, "Connect failed");
like ("@msg", qr{.*hopefully-doesnt-existst.*}, "Cannot open from non-existing directory with attributes in DSN");
@msg = ();
- eval {
- local $SIG{__DIE__} = sub { push @msg, @_ };
- my $dbh = DBI->connect ("dbi:DBM:", , undef, undef, {
+ $dbh = eval {
+ local $SIG{__WARN__} = sub { push @msg, @_ };
+ local $SIG{__DIE__} = sub { push @msg, @_ };
+ DBI->connect ("dbi:DBM:", , undef, undef, {
f_dir => "./hopefully-doesnt-existst",
sql_identifier_case => 1,
RaiseError => 1,
});
};
+ is ($dbh, undef, "Connect failed");
like ("@msg", qr{.*hopefully-doesnt-existst}, "Cannot open from non-existing directory with attributes in HASH");
}

11
perl-DBI.changes
View File

@ -1,3 +1,14 @@
-------------------------------------------------------------------
Thu Jan 9 19:10:11 UTC 2025 - Tina Müller <tina.mueller@suse.com>
- Remove perl-DBI-CVE-2014-10402.patch (fixed upstream)
-------------------------------------------------------------------
Wed Sep 4 05:31:43 UTC 2024 - Tina Müller <timueller+perl@suse.de>
- updated to 1.645.0 (1.645)
see /usr/share/doc/packages/perl-DBI/Changes
-------------------------------------------------------------------
Mon Mar 4 12:41:19 UTC 2024 - pgajdos@suse.com

111
perl-DBI.spec
View File

@ -1,7 +1,7 @@
#
# spec file for package perl-DBI
#
# Copyright (c) 2024 SUSE LLC
# Copyright (c) 2025 SUSE LLC
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@ -16,24 +16,105 @@
#
Name: perl-DBI
Version: 1.643
Release: 0
%define cpan_name DBI
Summary: Database independent interface for Perl
Name: perl-DBI
Version: 1.645.0
Release: 0
# 1.645 -> normalize -> 1.645.0
%define cpan_version 1.645
License: Artistic-1.0 OR GPL-1.0-or-later
Group: Development/Libraries/Perl
Summary: Database independent interface for Perl
URL: https://metacpan.org/release/%{cpan_name}
Source0: https://cpan.metacpan.org/authors/id/T/TI/TIMB/%{cpan_name}-%{version}.tar.gz
Source0: https://cpan.metacpan.org/authors/id/H/HM/HMBRAND/%{cpan_name}-%{cpan_version}.tgz
Source1: perl-DBI.rpmlintrc
Source2: cpanspec.yml
# MANUAL BEGIN
Patch0: perl-DBI-CVE-2014-10402.patch
# MANUAL END
BuildRoot: %{_tmppath}/%{name}-%{version}-build
BuildRequires: perl
BuildRequires: perl-macros
BuildRequires: perl(ExtUtils::MakeMaker) >= 6.48
BuildRequires: perl(Test::Simple) >= 0.90
Provides: perl(Bundle::DBI) = 12.008696
Provides: perl(DBD::DBM) = 0.08
Provides: perl(DBD::DBM::Statement)
Provides: perl(DBD::DBM::Table)
Provides: perl(DBD::DBM::db)
Provides: perl(DBD::DBM::dr)
Provides: perl(DBD::DBM::st)
Provides: perl(DBD::ExampleP) = 12.014311
Provides: perl(DBD::File) = 0.44
Provides: perl(DBD::File::DataSource::File)
Provides: perl(DBD::File::DataSource::Stream)
Provides: perl(DBD::File::Statement)
Provides: perl(DBD::File::Table)
Provides: perl(DBD::File::TableSource::FileSystem)
Provides: perl(DBD::File::db)
Provides: perl(DBD::File::dr)
Provides: perl(DBD::File::st)
Provides: perl(DBD::Gofer) = 0.015327
Provides: perl(DBD::Gofer::Policy::Base) = 0.010088
Provides: perl(DBD::Gofer::Policy::classic) = 0.010088
Provides: perl(DBD::Gofer::Policy::pedantic) = 0.010088
Provides: perl(DBD::Gofer::Policy::rush) = 0.010088
Provides: perl(DBD::Gofer::Transport::Base) = 0.014121
Provides: perl(DBD::Gofer::Transport::corostream)
Provides: perl(DBD::Gofer::Transport::null) = 0.010088
Provides: perl(DBD::Gofer::Transport::pipeone) = 0.010088
Provides: perl(DBD::Gofer::Transport::stream) = 0.014599
Provides: perl(DBD::Mem) = 0.001
Provides: perl(DBD::Mem::DataSource)
Provides: perl(DBD::Mem::Statement)
Provides: perl(DBD::Mem::Table)
Provides: perl(DBD::Mem::db)
Provides: perl(DBD::Mem::dr)
Provides: perl(DBD::Mem::st)
Provides: perl(DBD::NullP) = 12.014715
Provides: perl(DBD::Proxy) = 0.2004
Provides: perl(DBD::Proxy::db)
Provides: perl(DBD::Proxy::dr)
Provides: perl(DBD::Proxy::st)
Provides: perl(DBD::Sponge) = 12.010003
Provides: perl(DBDI)
Provides: perl(DBI) = %{version}
Provides: perl(DBI::Const::GetInfo::ANSI) = 2.008697
Provides: perl(DBI::Const::GetInfo::ODBC) = 2.011374
Provides: perl(DBI::Const::GetInfoReturn) = 2.008697
Provides: perl(DBI::Const::GetInfoType) = 2.008697
Provides: perl(DBI::DBD) = 12.015129
Provides: perl(DBI::DBD::Metadata) = 2.014214
Provides: perl(DBI::DBD::SqlEngine) = 0.06
Provides: perl(DBI::DBD::SqlEngine::DataSource)
Provides: perl(DBI::DBD::SqlEngine::Statement)
Provides: perl(DBI::DBD::SqlEngine::Table)
Provides: perl(DBI::DBD::SqlEngine::TableSource)
Provides: perl(DBI::DBD::SqlEngine::TieMeta)
Provides: perl(DBI::DBD::SqlEngine::TieTables)
Provides: perl(DBI::DBD::SqlEngine::db)
Provides: perl(DBI::DBD::SqlEngine::dr)
Provides: perl(DBI::DBD::SqlEngine::st)
Provides: perl(DBI::Gofer::Execute) = 0.014283
Provides: perl(DBI::Gofer::Request) = 0.012537
Provides: perl(DBI::Gofer::Response) = 0.011566
Provides: perl(DBI::Gofer::Serializer::Base) = 0.009950
Provides: perl(DBI::Gofer::Serializer::DataDumper) = 0.009950
Provides: perl(DBI::Gofer::Serializer::Storable) = 0.015586
Provides: perl(DBI::Gofer::Transport::Base) = 0.012537
Provides: perl(DBI::Gofer::Transport::pipeone) = 0.012537
Provides: perl(DBI::Gofer::Transport::stream) = 0.012537
Provides: perl(DBI::Profile) = 2.015065
Provides: perl(DBI::ProfileData) = 2.010008
Provides: perl(DBI::ProfileDumper) = 2.015325
Provides: perl(DBI::ProfileDumper::Apache) = 2.014121
Provides: perl(DBI::ProfileSubs) = 0.009396
Provides: perl(DBI::ProxyServer) = 0.3005
Provides: perl(DBI::ProxyServer::db)
Provides: perl(DBI::ProxyServer::dr)
Provides: perl(DBI::ProxyServer::st)
Provides: perl(DBI::SQL::Nano) = 1.015544
Provides: perl(DBI::SQL::Nano::Statement_)
Provides: perl(DBI::SQL::Nano::Table_)
Provides: perl(DBI::Util::CacheMemory) = 0.010315
Provides: perl(DBI::Util::_accessor) = 0.009479
Provides: perl(DBI::common)
%undefine __perllib_provides
%{perl_requires}
%description
@ -54,12 +135,13 @@ at least the following perl manuals first: perlreftut, perldsc, perllol,
and perlboot.
%prep
%autosetup -p1 -n %{cpan_name}-%{version}
find . -type f ! -path "*/t/*" ! -name "*.pl" ! -path "*/bin/*" ! -path "*/script/*" ! -name "configure" -print0 | xargs -0 chmod 644
%autosetup -n %{cpan_name}-%{cpan_version}
find . -type f ! -path "*/t/*" ! -name "*.pl" ! -path "*/bin/*" ! -path "*/script/*" ! -path "*/scripts/*" ! -name "configure" -print0 | xargs -0 chmod 644
%build
perl Makefile.PL INSTALLDIRS=vendor OPTIMIZE="%{optflags}"
make %{?_smp_mflags}
%make_build
%check
make test
@ -70,7 +152,6 @@ make test
%perl_gen_filelist
%files -f %{name}.files
%defattr(-,root,root,755)
%doc Changes Driver.xst README.md
%license LICENSE