[SERVER-30428] Windows core dump generation is broken Created: 31/Jul/17 Updated: 30/Oct/23 Resolved: 30/Aug/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | 3.5.10 |
| Fix Version/s: | 3.5.11 |
| Type: | Bug | Priority: | Critical - P2 |
| Reporter: | Kaloian Manassiev | Assignee: | Robert Guo (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Participants: | |||||||||
| Description |
|
The core dump generation code on Windows does an incorrect string concatenation which prevents it from writing the file. The error is shown below. It looks like an underscore is inserted before the dump file name:
|
| Comments |
| Comment by Robert Guo (Inactive) [ 31/Jul/17 ] |
|
Reverted the offending commit here. Closing this ticket as Done. We'll take Mark's suggestion into account for subsequent work. |
| Comment by Mark Benvenuto [ 31/Jul/17 ] |
|
Assigning to TIG since they introduced the regression and raising the priority since it blocks debugging. Please revert the breaking change. I recommend verifying the fix by introducing an access violation somewhere in mongod.exe, and verifying the dump can be opened with WinDBG. |
| Comment by Kaloian Manassiev [ 31/Jul/17 ] |
|
I would appreciate it if we do that, because currently this makes investigating what would have been simple crashes on Windows impossible and requires me to attach the debugger. |
| Comment by Max Hirschhorn [ 31/Jul/17 ] |
|
I think we should just revert the changes from SERVER-29424 until we're able to correctly handle path concatenation. The dump_ prefix is intended to be for the core dump's basename and not a prefix for the entire path. CC robert.guo, mark.benvenuto |