io: Avoid infinite recursion in TFile::Open

This fixes root-project#15590
pcanal · May 21, 2024 · ddecc0d · ddecc0d
1 parent a2d3663
commit ddecc0d
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/io/io/src/TFile.cxx b/io/io/src/TFile.cxx
@@ -4285,16 +4285,17 @@ TFile *TFile::Open(const char *url, Option_t *options, const char *ftitle,
                   f = (TFile*) h->ExecPlugin(4, name.Data(), option, ftitle, compress);
             } else {
                // Just try to open it locally but via TFile::Open, so that we pick-up the correct
-               // plug-in in the case file name contains information about a special backend (e.g.
-               f = TFile::Open(urlname.GetFileAndOptions(), option, ftitle, compress);
+               // plug-in in the case file name contains information about a special backend (e.g.)
+               if (strcmp(name, urlname.GetFileAndOptions()) != 0)
+                  f = TFile::Open(urlname.GetFileAndOptions(), option, ftitle, compress);
             }
          }
       }
 
       if (f && f->IsZombie()) {
          TString newUrl = f->GetNewUrl();
          delete f;
-         if( newUrl.Length() && gEnv->GetValue("TFile.CrossProtocolRedirects", 1) )
+         if( newUrl.Length() && (newUrl != name) && gEnv->GetValue("TFile.CrossProtocolRedirects", 1) )
             f = TFile::Open( newUrl, option, ftitle, compress );
          else
             f = nullptr;