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
pure perl assertion failure with POSIX::sigaction #10385
Comments
From @nwc10Created by @nwc10This little test case: #!/usr/local/bin/perl -w use strict; use POSIX ':signal_h' ; # This is the special bit *SIG = {}; my $mask = POSIX::SigSet->new(); sigaction(SIGCHLD, $act); __END__ fails an assertion: perl5.13.0: mg.c:98: S_save_magic: Assertion `((sv)->sv_flags & (0x00200000|0x00400000|0x00800000))' failed. As best I can tell, that assertion was added in 1996 as part of perl5.002beta3 The test case is massively reduced from part of the current forks distribution. I don't know what the right fix is. This doesn't seem to be a core regression. Nicholas Clark Perl Info
|
From @nwc10On Mon, May 17, 2010 at 02:10:14PM -0700, Nicholas Clark wrote:
Although there is this highly suspicious comment in the call stack: /* This call actually calls sigaction() with almost the Nicholas Clark |
From @ntyniThis is a bug report for perl from Niko Tyni <ntyni@debian.org>, Originally triggered by the CPAN forks module: ./perl -Ilib -MPOSIX -e '*SIG={}; sigaction(1, POSIX::SigAction->new)' Seen at least on 5.10.1 and blead. ( This is also http://bugs.debian.org/594250 ) Flags: Site configuration information for perl 5.13.4: Configured by niko at Wed Aug 25 00:21:02 EEST 2010. Summary of my perl5 (revision 5 version 13 subversion 4) configuration: Locally applied patches: @INC for perl 5.13.4: Environment for perl 5.13.4: |
From @tonycozOn Tue, Aug 24, 2010 at 10:32:18PM -0700, Niko Tyni wrote:
Fixed in 17cffb3. Tony |
The RT System itself - Status changed from 'new' to 'open' |
@tonycoz - Status changed from 'open' to 'resolved' |
From @nwc10On Tue, Aug 31, 2010 at 12:41:29AM +1000, Tony Cook wrote:
I believe that this bug is a duplicate of 75150 Whilst this fixes the immediate symptoms of an assertion failure, I'm not *SIG={} Nicholas Clark |
From @tonycozOn Mon, Aug 30, 2010 at 04:01:28PM +0100, Nicholas Clark wrote:
The patch prevents the test code from 75150 crashing.
I'm not sure the current behaviour is insane. Tony |
The RT System itself - Status changed from 'new' to 'open' |
@iabyn - Status changed from 'open' to 'resolved' |
@nwc10 - Status changed from 'resolved' to 'open' |
From @nwc10On Tue Aug 31 02:25:47 2010, tonyc wrote:
Too many negatives. I'm parsing that as "It may not be wrong" I don't consider this resolved yet. Nicholas Clark |
From zefram@fysh.orgNicholas Clark wrote:
The current behaviour looks OK to me. The signal handler doesn't get Do you have a specific proposal of what the behaviour should be? If not, -zefram |
Migrated from rt.perl.org#75150 (status was 'open')
Searchable as RT75150$
The text was updated successfully, but these errors were encountered: