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
assertion failure fiddling with @ISA #9337
Comments
From @ntyniThis is a bug report for perl from Niko Tyni <ntyni@debian.org>, This one-liner, reduced from code in HTML::Mason::Request, causes an % ./miniperl -e '@ISA=q/a/; $#ISA; $ISA[0]=q/b/' #0 0x00002b6aba3ac165 in raise () from /lib/libc.so.6 As seen in <http://bugs.debian.org/480480>, this segfaults on 5.10.0 The segfault is gone with change 33684, but the assertion failure remains Flags: Site configuration information for perl 5.10.0: Configured by Debian Project at Tue May 20 20:09:17 UTC 2008. Summary of my perl5 (revision 5 version 10 subversion 0) configuration: Locally applied patches: @INC for perl 5.10.0: Environment for perl 5.10.0: |
From @rgs2008/5/21 via RT Niko Tyni <perlbug-followup@perl.org>:
Looks like the original author of the code overlooked the possibility Here's a patch : Inline Patch--- mg.c.orig 2008-05-12 18:29:57.000000000 +0200
+++ mg.c 2008-05-21 14:23:45.000000000 +0200
@@ -1583,7 +1583,7 @@ Perl_magic_setisa(pTHX_ SV *sv, MAGIC *m
stash = GvSTASH(
SvTYPE(mg->mg_obj) == SVt_PVGV
? (GV*)mg->mg_obj
- : (GV*)SvMAGIC(mg->mg_obj)->mg_obj
+ : (GV*)mg_find(mg->mg_obj, PERL_MAGIC_isa)->mg_obj
);
if (stash)
@@ -1609,7 +1609,7 @@ Perl_magic_clearisa(pTHX_ SV *sv, MAGIC
stash = GvSTASH(
SvTYPE(mg->mg_obj) == SVt_PVGV
? (GV*)mg->mg_obj
- : (GV*)SvMAGIC(mg->mg_obj)->mg_obj
+ : (GV*)mg_find(mg->mg_obj, PERL_MAGIC_isa)->mg_obj
);
if (stash) |
The RT System itself - Status changed from 'new' to 'open' |
From @TuxOn Wed, 21 May 2008 14:42:48 +0200, "Rafael Garcia-Suarez"
Change 33898 submitted.
-- |
@rgs - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#54566 (status was 'resolved')
Searchable as RT54566$
The text was updated successfully, but these errors were encountered: