[SERVER-11002] Insufficient permission will crash checkReadAhead() Created: 02/Oct/13 Updated: 11/Jul/16 Resolved: 07/Oct/13 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Logging |
| Affects Version/s: | None |
| Fix Version/s: | 2.5.3 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Christian Zuckschwerdt | Assignee: | Matt Kangas |
| Resolution: | Done | Votes: | 0 |
| Labels: | pull-request | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Linux with Grsec / VM |
||
| Backwards Compatibility: | Fully Compatible |
| Operating System: | Linux |
| Participants: |
| Description |
|
In https://github.com/mongodb/mongo/blob/master/src/mongo/db/db.cpp#L614 An exception will be thrown if the path can't be accessed (due to insufficient access rights in my case). This will crash the startup. I gather that checking the readahead is informational only. It would be an improvement to just log the error and carry on. The attached patch will catch the filesystem exception and log a line of See here for the changes: And here for the complete patch with indentation |
| Comments |
| Comment by Matt Kangas [ 07/Oct/13 ] |
|
Merged in 69079a7 and b766095. Thanks for contributing to MongoDB! |
| Comment by Matt Kangas [ 03/Oct/13 ] |
|
Hi Christian, Your patch looks good. If you sign our contributor's agreement and submit this as a pull request, I will be happy to give you full credit for the fix.
Please include " |