-
Notifications
You must be signed in to change notification settings - Fork 550
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
Regular expression bug (5.00561) #477
Comments
From ed_peschko@csgsystems.com$line = ('a' x 99999) . "'" . ('a' x 20000); (in other words, if one is looking for a single string with possible Ed |
From @schwern(From perlbug 19990906.001)
Below is a patch to t/run/kill_perl.t to ensure it doesn't creep in Bug closed. --- t/run/kill_perl.t 2001/10/30 03:31:32 1.1 -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
1 similar comment
From @schwern(From perlbug 19990906.001)
Below is a patch to t/run/kill_perl.t to ensure it doesn't creep in Bug closed. --- t/run/kill_perl.t 2001/10/30 03:31:32 1.1 -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From @jhiThanks, applied. Though I still think someone should go in with a -- |
From @schwernExcept that !(length $test % 73) doesn't cause the rest of the tests to
Yes, it is. However as the bug is still in flux, having flip-flopped kill_perl.t is a dumping ground, but when you want to write a quick But, like any compost heap, it needs turning over once in a while. My 1) Start at the top of the list (presumably those are the oldest). I suspect the majority will turn out to not be segfaulting on anything -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From [Unknown Contact. See original ticket]$line = ('a' x 99999) . "'" . ('a' x 20000); (in other words, if one is looking for a single string with possible Ed |
From @jhiTalk with Nick Clark about arithmetic operators like % and you'll -- |
From @andk> Thanks, applied. It seems you're talking 12776. I didn't see Schwern's posting, but t/run/kill_perl.....................# PROG: -- |
From @schwernSorry, ran diff with brain in neutral. THIS is the proper patch. --- t/run/kill_perl.t 2001/10/30 03:31:32 1.1 -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From @jhiArgh. True. And if I dig up the *original* lines from the $line = ('a' x 99999) . "'" . ('a' x 20000); we still get a stack overflow core.
-- |
From @jhiYup, and that still blows off the stack.
-- |
From @schwernOdd, doesn't for me. perl -V follows. Here's a variant of that test patch which makes this test TODO. --- t/run/kill_perl.t 2001/10/30 03:31:32 1.1 my @prgs = (); my $test = 1; my $switch; @@ -809,3 +814,10 @@ Summary of my perl5 (revision 5.0 version 7 subversion 2 patch 12773) configuration: Characteristics of this binary (from libperl): -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From @jhiDoes for me in tru64 and linux/x86. Solaris/sparc doesn't. But hmmm... yup, dropping the stack limit in Solaris to 7MB (from 8MB) Lesson: absence of evidence does not evidence of absence make.
Hmmm, I hope we have core dump cleanup (and not just "core") going on -- |
From @schwernOk, extending the size of the string causes it to core here. --- t/run/kill_perl.t 2001/10/30 05:35:40 1.2 -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From [Unknown Contact. See original ticket]$line = ('a' x 99999) . "'" . ('a' x 20000); (in other words, if one is looking for a single string with possible Ed |
From @schwern$line = ('a' x 999999) . "'" . ('a' x 200000); -- Michael G. Schwern <schwern@pobox.com> http://www.pobox.com/~schwern/ |
From @nwc10Are there just 3 classes of tests? 1: Things that go wrong that really need a clean new copy of perl run to So do we want a kill_perl and a clean_perl test, and sweep out all the Along with comments for each subtest justifying why it needs the slow startup Nicholas Clark |
From @jhiI'm still not happy with including this test. Even though t/TEST t/run/kill_perl.....................Stack overflow: pid 14353, proc perl, addr 0x11fdfffb0, pc 0x3ffbff787f0 ok And I'm certain other places will have similarly unpleasant displays. Yes, I know it's slightly odd from QA viewpoint to refuse not to But what I am to do? Include few dozen tests that core dump with Maybe we should have a new (non-.t) dumping ground for these tests, that
-- |
From @jhiSounds about right.
-- |
From @nwc10This makes it seem reasonable:
#!./perl BEGIN { use strict; BEGIN { and then run each of them in a separate child perl interpreter. Nicholas Clark |
From [Unknown Contact. See original ticket]The child interpreter land does have great promises for Safe in the future, but I wouldn't know how to catch segfaults! They tend to Arthur |
From @jhiThat could work. -- |
From @nwc10
Sorry, wasn't clear. The idea was that every test would run, so that harness Nicholas Clark |
From [Unknown Contact. See original ticket]Perhaps even turn it on by default if -Dusedevel, so that -- BKS __________________________________________________ |
From @jhiYes, reusing (or extending) the $Config{usedevel} for this -- |
From @nwc10usedevel didn't seem to be in config.sh when I had a look on one of my Nicholas Clark |
@rspier - Status changed from 'open' to 'resolved' |
From The RT System itselfreplicated on linux running bleadperl DEVEL7093 |
Migrated from rt.perl.org#1318 (status was 'resolved')
Searchable as RT1318$
The text was updated successfully, but these errors were encountered: