This should only be merged on top of #505
[XrdCeph] Fixed some resource leak when Posix_Open fails
File descriptor were allocated (this is not a leak per se, just a counter increasing) and stored into maps where they would never be deleted in case of error.
Now maps are cleaned up in case of errors
You can view, comment on, or merge this pull request online at:
https://github.com/xrootd/xrootd/pull/506
Commit Summary
- [XrdCeph] Added support for namelib in ceph plugin
- [xrdCeph] Fixed internal (f)stat so that it sets S_IFREG in returned mode
- [xrdCeph] properly return ENOENT when file does not exist in open for read
- [xrdceph] Fixed configuration of the XrdCephOss module
- [XrdCeph] Fixed some resource leak when Posix_Open fails
File Changes
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":"Fd leak fix (#506)"}],"action":{"name":"View Pull Request","url":"https://github.com/xrootd/xrootd/pull/506"}}}
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