Skip to content

Commit db69612

Browse files
committed
sv_setsv_cow: apply the same tests we do for a normal COW copy
Previously if you had a successful match against an SV with a SvLEN() large relative to the SvCUR() the regexp engine would use sv_setsv_cow() to make a COW copy of the matched SV, extending the life of the large allocation buffer. A normal sv_setsv() normally didn't do such a COW copy, but the above also marked the source SV as COW, so further copies of the SV could even further extend the lifetime of the buffer, eg: while (<>) { # readline tends to make large SvLEN() /something/; # some sort of match push @save, $_; # with a successful match, the large $_ buffer # survives until @save is released } Fixes part of Perl#21877
1 parent f71cf50 commit db69612

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

sv.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4912,6 +4912,12 @@ Perl_sv_setsv_cow(pTHX_ SV **pdsv, SV *ssv)
49124912
sv_dump(dsv);
49134913
}
49144914
#endif
4915+
if (!CHECK_COWBUF_THRESHOLD(cur, len)
4916+
|| ! CHECK_COW_THRESHOLD(cur, len)) {
4917+
DEBUG_C(PerlIO_printf(Perl_debug_log,
4918+
"Fast copy on write: Sizes not appropriate to COW\n"));
4919+
return FALSE;
4920+
}
49154921
if (dsv) {
49164922
if (SvTHINKFIRST(dsv))
49174923
sv_force_normal_flags(dsv, SV_COW_DROP_PV);

0 commit comments

Comments
 (0)