[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:
Related
is related to SERVER-29424 Make Windows minidump file naming con... Backlog
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:

...
c20002| 2017-07-31T10:56:40.905-0400 I -        [rsSync]
c20002| 2017-07-31T10:56:40.905-0400 I CONTROL  [rsSync] failed to open minidump file dump_E:\workspace\mongo\mongod.17064.2017-07-31T14-56-40.mdmp : The filename, directory name, or volume label syntax is incorrect.
c20002| 2017-07-31T10:56:40.905-0400 I CONTROL  [rsSync] *** immediate exit due to unhandled exception



 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

Generated at Thu Feb 08 04:23:47 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.