[XrdCl] LocalFileHandler: be more robust in case PostMaster initializ…

…ation failed, closes #1797.
xrootd · Oct 7, 2022 · 32ada89 · 32ada89
1 parent ec959e0
commit 32ada89
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/src/XrdCl/XrdClLocalFileHandler.cc b/src/XrdCl/XrdClLocalFileHandler.cc
@@ -201,7 +201,7 @@ namespace
         // and return there is no need to execute this in the thread-pool
         SyncResponseHandler *syncHandler =
             dynamic_cast<SyncResponseHandler*>( handler );
-        if( syncHandler )
+        if( syncHandler || DefaultEnv::GetPostMaster() == nullptr )
         {
           syncHandler->HandleResponse( status, resp );
         }
@@ -230,7 +230,6 @@ namespace XrdCl
   LocalFileHandler::LocalFileHandler() :
       fd( -1 )
   {
-    jmngr = DefaultEnv::GetPostMaster()->GetJobManager();
   }
 
   //------------------------------------------------------------------------
@@ -767,15 +766,15 @@ namespace XrdCl
     // and return there is no need to execute this in the thread-pool
     SyncResponseHandler *syncHandler =
         dynamic_cast<SyncResponseHandler*>( handler );
-    if( syncHandler )
+    if( syncHandler || DefaultEnv::GetPostMaster() == nullptr )
     {
       syncHandler->HandleResponse( st, resp );
       return XRootDStatus();
     }
 
     HostList *hosts = pHostList.empty() ? 0 : new HostList( pHostList );
     LocalFileTask *task = new LocalFileTask( st, resp, hosts, handler );
-    jmngr->QueueJob( task );
+    DefaultEnv::GetPostMaster()->GetJobManager()->QueueJob( task );
     return XRootDStatus();
   }
 

diff --git a/src/XrdCl/XrdClLocalFileHandler.hh b/src/XrdCl/XrdClLocalFileHandler.hh
@@ -330,11 +330,6 @@ namespace XrdCl
                               char             *body,
                               ResponseHandler  *handler );
 
-      //---------------------------------------------------------------------
-      // Receives LocalFileTasks to handle them async
-      //---------------------------------------------------------------------
-      JobManager *jmngr;
-
       //---------------------------------------------------------------------
       // Internal filedescriptor, which is used by all operations after open
       //---------------------------------------------------------------------