New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Commit eb06eac93f0 is asking for trouble. #10643
Comments
From @cpansproutIt does not actually introduce a bug, as far as I can tell, but it has the potential to make future refactorings cause SEGVs, as it uses PL_op without checking it first. The attached patch rectifies this. |
From @cpansproutFrom: Father Chrysostomos <sprout@cpan.org> Don’t use PL_op without checking it first. Inline Patch--- blead-45549-warn3/doio.c.orig 2010-08-08 18:25:47.000000000 -0700
+++ blead-45549-warn3/doio.c 2010-09-16 08:33:07.000000000 -0700
@@ -1241,7 +1241,9 @@ Perl_do_print(pTHX_ register SV *sv, Per
else {
assert((char *)result == tmps);
Perl_ck_warner_d(aTHX_ packWARN(WARN_UTF8),
- "Wide character in %s", OP_DESC(PL_op));
+ "Wide character in %s",
+ PL_op ? OP_DESC(PL_op) : "print"
+ );
}
}
/* To detect whether the process is about to overstep its |
@rgs - Status changed from 'new' to 'resolved' |
From @rgarciaOn 19 September 2010 21:22, Father Chrysostomos
Thanks, applied to bleadperl. |
Migrated from rt.perl.org#77932 (status was 'resolved')
Searchable as RT77932$
The text was updated successfully, but these errors were encountered: