Bug 720610
| Summary: | [PATCH] large string repeat count causes heap corruption | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jim Meyering <meyering> | ||||||
| Component: | perl | Assignee: | Petr Pisar <ppisar> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | unspecified | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | 15 | CC: | cweyl, iarnell, kasal, lkundrak, mmaslano, ppisar, psabata, rc040203, tcallawa | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | perl-5.12.4-148.fc14 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | |||||||||
| : | 720644 720652 (view as bug list) | Environment: | |||||||
| Last Closed: | 2011-11-13 05:36:03 UTC | Type: | --- | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Attachments: |
|
||||||||
|
Description
Jim Meyering
2011-07-12 09:20:49 UTC
F14 (perl-5.12.3-143.fc14.x86_64) affected too. Am I right this changes public API, thus without rebuilding all perl packages we cannot apply it? I made a very quick test in rawhide. After applying this patch and rebuilding perl-5.14.2, I downloaded a few XS dists at random and ran their test suites against the our existing packaged version with no failures. I don't know of any modules actually using repeatcpy, nor do I have a local cpan to grep for one. Looking through matches from here, https://codesearch.google.com/#search/&q=repeatcpy%5B%5E%7C%5D&p=1&type=cs I found only one use: Regexp-Fields-0.046/sources/regcomp-5.6.1.c 1094: SvGROW(last_str, (mincount * l) + 1); 1095: repeatcpy(SvPVX(last_str) + l, 1096: SvPVX(last_str), l, mincount - 1); Ok, I'll commit it into all branches with your IV type which is signed as well as original I32. Created attachment 526740 [details]
Patch for Perl 5.14.2
This patch for 5.14.2 modifies embed.fnc instead of proto.h as the later one is generated from the first one (by regen.pl script in root of Perl sources). Otherwise standard test t/porting/regen.t fails.
perl-5.14.2-189.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-5.14.2-189.fc16 Package perl-5.14.2-189.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing perl-5.14.2-189.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-15276 then log in and leave karma (feedback). Package perl-5.14.2-190.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing perl-5.14.2-190.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-15276 then log in and leave karma (feedback). perl-5.12.4-163.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-5.12.4-163.fc15 perl-5.12.4-148.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/perl-5.12.4-148.fc14 perl-5.14.2-190.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. perl-5.12.4-163.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report. perl-5.12.4-148.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. |