Lock m_is_open in File::ioActive().

Fixes crash in the following scenario:

a) Client logs in and opens a file.
b) Starts reading.
c) The base file is either created or already exists from a previous open but there is no cinfo file as of yet.
d) Client crashes (I/O may or may not be active as of yet but likely is).
e) XRootD closes the file and close call IOActive() to see if the file can be closed.
f) The FileCache code goes through it's pending block chain to see which need to be removed. However, the data structure is still in the process of being built and it encounters a partially completed entry and crashes on that one.

You can view, comment on, or merge this pull request online at:

https://github.com/xrootd/xrootd/pull/477

Commit Summary

Check m_is_open state under lock.

File Changes

M src/XrdFileCache/XrdFileCacheFile.cc (67)

Patch Links:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"pfc: Handle client disconnect at the startup time (#477)"}],"action":{"name":"View Pull Request","url":"https://github.com/xrootd/xrootd/pull/477"}}}

Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1