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
Bleadperl v5.19.3-96-ge8eb279 breaks MLEHMANN/Coro-6.37.tar.gz #13784
Comments
From @andkgit bisect e8eb279 is the first bad commit Use SSize_t for tmps stack offsets This is a partial fix for #119161. On 64-bit platforms, I32 is too small to hold offsets into a stack With this commit, ()=1..1000000000000 stops crashing immediately. diagnostics http://www.cpantesters.org/cpan/report/f43d6962-cf48-11e3-a383-9294310f0ce5 perl -V Summary of my perl5 (revision 5 version 19 subversion 4) configuration: Characteristics of this binary (from libperl): -- |
From @iabynOn Tue, Apr 29, 2014 at 08:34:29PM -0700, Andreas J. Koenig via RT wrote:
This is because Coro declares a structure that mimics a perl interpreter With this diff: -6.37-/Coro/state.h Coro-6.37/Coro/state.h Inline Patch--- Coro-6.37-/Coro/state.h 2012-02-20 07:51:30.000000000 +0000
+++ Coro-6.37/Coro/state.h 2014-04-30 14:31:15.952377627 +0100
@@ -37,9 +37,9 @@
VAR(savestack_max, I32)
VAR(tmps_stack, SV **) /* mortals we've made */
-VAR(tmps_ix, I32)
-VAR(tmps_floor, I32)
-VAR(tmps_max, I32)
+VAR(tmps_ix, SSize_t)
+VAR(tmps_floor, SSize_t)
+VAR(tmps_max, SSize_t)
VAR(markstack, I32 *) /* stack_sp locations we're remembering */
VAR(markstack_p
As for the t/19_handle.t SEGV, slf_init_rw() in Ev.xs is trying to -- |
The RT System itself - Status changed from 'new' to 'open' |
From @rjbsThis was corrected in 6.39. -- |
@rjbs - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#121767 (status was 'resolved')
Searchable as RT121767$
The text was updated successfully, but these errors were encountered: