URL: <http://savannah.cern.ch/bugs/?78973> Summary: xproofd segfaulting in the xrootd fork handler Project: XROOTD Submitted by: ljanyst Submitted on: 2011-03-03 15:28 Category: None Severity: 3 - Normal Priority: 5 - Normal Item Group: None Status: None Privacy: Public Assigned to: ljanyst Originator Email: Open/Closed: Open Discussion Lock: Any _______________________________________________________ Details: ++ CUT ++ On Wed, Mar 02, 2011 at 06:01:53PM -0600, Chris Jones wrote: In our xrootd5.27.06b-cms3 version of xrootd we added a modification from the xrootd developers to deal with 'fork'ing. However, when xproofd does its fork a pre-condition expected by the xrootd code does not hold and the code tries to talk to a null C++ object. However, I think this could easily be fixed by changing the file XrdClient/XrdClientEnv.cc such that before every call to 'ConnectionManager' the code first does if( 0 != ConnectionManager ) { .... } -- CUT -- ++ CUT ++ 2011/3/3 Lukasz Janyst <[log in to unmask]>: This is one of the reasons why the final version of the patch that has made it to the xroot repository is sligthly different that the one you got: the fork handlers are disabled by default and one can enable them by setting an environment flag: http://xrootd.cern.ch/cgi-bin/cgit.cgi/xrootd/patch/?id=953079e8605f58f0500e6b4ed774a8ed91a75671 I can also add the check suggested by Chris just to be extra careful. -- CUT -- ++ CUT ++ 2011/3/3 Peter Elmer <[log in to unmask]>: Ok, it looks like the environment for this fix is something we can turn on programmatically (and not via an envvar, which we don't use for configuration), is that correct? What is the reason for making it disabled by default? Is there a penalty for using it in the non-fork cases or is this just "caution" because it is a new/experimental feature? -- CUT -- _______________________________________________________ Reply to this item at: <http://savannah.cern.ch/bugs/?78973> _______________________________________________ Message sent via/by LCG Savannah http://savannah.cern.ch/