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
Core dump in bleadperl from perl -e '$! = 9999; $a = $!;' #10178
Comments
From @timbunceperl -e '$! = 9999; #0 0x0012e008 in Perl_magic_get (my_perl=0x1000000, sv=0x10489f0, mg=0x847930) at mg.c:1055 (gdb) print *sv Perl Info
|
From @rgarciaOn 15 February 2010 15:06, Tim Bunce <perlbug-followup@perl.org> wrote:
I do not reproduce this ; what does strerror return for errno 9999 on |
The RT System itself - Status changed from 'new' to 'open' |
From @nwc10On Mon, Feb 15, 2010 at 06:06:50AM -0800, Tim Bunce wrote:
0x4407 PVMG, POK, pPOK
NULL. That's naughty. That SV is not well formed. I wonder, how did that come Try the appended patch. Tested on Linux, but not OS X
Nicholas Clark Inline Patchdiff --git a/mg.c b/mg.c
index cc01547..06c899e 100644
--- a/mg.c
+++ b/mg.c
@@ -1048,7 +1048,8 @@ Perl_magic_get(pTHX_ SV *sv, MAGIC *mg)
else
#endif
sv_setpv(sv, errno ? Strerror(errno) : "");
- SvPOK_on(sv); /* may have got removed during taint processing */
+ if (SvPOKp(sv))
+ SvPOK_on(sv); /* may have got removed during taint processing */
RESTORE_ERRNO;
}
|
From @smpetersOn Tue, Feb 16, 2010 at 3:59 AM, Nicholas Clark <nick@ccl4.org> wrote:
That fixes it for me on OS X. Steve |
From @smpetersOn Tue, Feb 16, 2010 at 9:12 AM, Steve Peters <steve@fisharerojo.org> wrote:
Here's a test case for the fix as well. Steve --- a/t/op/magic.t -plan (tests => 79);
|
From @nwc10On Tue, Feb 16, 2010 at 09:25:21AM -0600, Steve Peters wrote:
Thanks, applied as part of be1cf43 Nicholas Clark |
@nwc10 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#72850 (status was 'resolved')
Searchable as RT72850$
The text was updated successfully, but these errors were encountered: