Skip to content
This repository was archived by the owner on Jun 1, 2023. It is now read-only.

Commit 1e739db

Browse files
author
Reini Urban
committed
XSLoader: fix new failing cperl test
The goto corrupts the CX stack #167. unrelated to the security fix #184.
1 parent 35c912b commit 1e739db

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

ext/DynaLoader/t/XSLoader.t

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -136,12 +136,12 @@ SKIP: {
136136
open my $fh,
137137
">$name/auto/Foo/Bar/Bar.$Config::Config{'dlext'}";
138138
close $fh;
139-
chmod 0755, $fname;
140139
my $fell_back;
140+
#my $cperl = 1 if $Config{usecperl};
141141
local *XSLoader::bootstrap_inherit = sub {
142142
$fell_back++;
143143
# Break out of the calling subs
144-
goto the_test;
144+
#goto the_test unless $cperl;
145145
};
146146
# https://rt.cpan.org/Ticket/Display.html?id=115808
147147
eval <<END;
@@ -152,6 +152,5 @@ END
152152
the_test:
153153
ok $fell_back,
154154
'XSLoader will not load relative paths based on (caller)[1]';
155-
sleep(0.2);
156155
File::Path::rmtree($name);
157156
}

0 commit comments

Comments
 (0)