This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
AutoLoader fix, part 2
authorSteffen Müller <0mgwtfbbq@sneakemail.com>
Wed, 10 Jan 2007 18:52:23 +0000 (19:52 +0100)
committerRafael Garcia-Suarez <rgarciasuarez@gmail.com>
Thu, 11 Jan 2007 09:51:56 +0000 (09:51 +0000)
Message-ID: <20070110175148.26694.qmail@lists.develooper.com>

p4raw-id: //depot/perl@29750

lib/AutoLoader.pm
lib/AutoLoader.t

index 25b3928..e33a4be 100644 (file)
@@ -102,7 +102,7 @@ sub find_filename {
            # (and failing) to find the 'lib/auto/foo/bar.al' because it
            # looked for 'lib/lib/auto/foo/bar.al', given @INC = ('lib').
 
-           if (-r $filename) {
+           if (defined $filename and -r $filename) {
                unless ($filename =~ m|^/|s) {
                    if ($is_dosish) {
                        unless ($filename =~ m{^([a-z]:)?[\\/]}is) {
index da7071b..92d66fa 100755 (executable)
@@ -16,7 +16,7 @@ BEGIN
        unshift @INC, $dir;
 }
 
-use Test::More tests => 21;
+use Test::More tests => 22;
 
 # First we must set up some autoloader files
 my $fulldir = File::Spec->catdir( $dir, 'auto', 'Foo' );
@@ -164,8 +164,21 @@ AutoLoader->unimport();
 
 ::is( Baz->AUTOLOAD(), 'i am here', '... but not non-imported AUTOLOAD()' );
 
+
+package SomeClass;
+use AutoLoader 'AUTOLOAD';
+sub new {
+    bless {} => shift;
+}
+
 package main;
 
+$INC{"SomeClass.pm"} = $0; # Prepare possible recursion
+{
+    my $p = SomeClass->new();
+} # <-- deep recursion in AUTOLOAD looking for SomeClass::DESTROY?
+::ok(1, "AutoLoader shouldn't loop forever if \%INC is modified");
+
 # cleanup
 END {
        return unless $dir && -d $dir;