aboutsummaryrefslogtreecommitdiffstats
path: root/main/perl-digest-sha1/perl-digest-sha1-check-object.patch
diff options
context:
space:
mode:
authorNatanael Copa <ncopa@alpinelinux.org>2014-07-15 07:09:03 +0000
committerNatanael Copa <ncopa@alpinelinux.org>2014-07-15 09:33:37 +0000
commit9cf168cc006e5c57929b47955447f44c065888e6 (patch)
treefa36eda6eeca115b0425fa4ce29289b34e86f0ea /main/perl-digest-sha1/perl-digest-sha1-check-object.patch
parenta286ee9c925bd1ae64378020f0fb6af1372ca69d (diff)
downloadaports-9cf168cc006e5c57929b47955447f44c065888e6.tar.bz2
aports-9cf168cc006e5c57929b47955447f44c065888e6.tar.xz
main/perl-digest-sha1: fix segfault
https://rt.cpan.org/Public/Bug/Display.html?id=94188
Diffstat (limited to 'main/perl-digest-sha1/perl-digest-sha1-check-object.patch')
-rw-r--r--main/perl-digest-sha1/perl-digest-sha1-check-object.patch22
1 files changed, 22 insertions, 0 deletions
diff --git a/main/perl-digest-sha1/perl-digest-sha1-check-object.patch b/main/perl-digest-sha1/perl-digest-sha1-check-object.patch
new file mode 100644
index 0000000000..110ef42d7e
--- /dev/null
+++ b/main/perl-digest-sha1/perl-digest-sha1-check-object.patch
@@ -0,0 +1,22 @@
+https://rt.cpan.org/Public/Ticket/Attachment/WithHeaders/712715
+
+The get_sha_info() function in SHA1.xs does not check that its argument
+is an actual object. This means that segfaults can be generated by
+commands such as:
+
+$ perl -Mblib -e "use Digest::SHA1; print Digest::SHA1->add(q(a))->hexdigest"
+Segmentation fault
+
+diff -Naur Digest-SHA1-2.13/SHA1.xs Digest-SHA1-2.13.patched/SHA1.xs
+--- Digest-SHA1-2.13/SHA1.xs 2010-07-02 23:51:12.000000000 -0700
++++ Digest-SHA1-2.13.patched/SHA1.xs 2014-03-25 12:43:53.233272555 -0700
+@@ -372,7 +372,7 @@
+
+ static SHA_INFO* get_sha_info(pTHX_ SV* sv)
+ {
+- if (sv_derived_from(sv, "Digest::SHA1"))
++ if (sv_isobject(sv) && sv_derived_from(sv, "Digest::SHA1"))
+ return INT2PTR(SHA_INFO*, SvIV(SvRV(sv)));
+ croak("Not a reference to a Digest::SHA1 object");
+ return (SHA_INFO*)0; /* some compilers insist on a return value */
+