Closed Bug 1332147 Opened 6 years ago Closed 6 years ago

Some minidumps cannot be loaded in VS, showing "xul.dll not found"


(Toolkit :: Crash Reporting, defect)

Not set



Tracking Status
firefox53 --- fixed
firefox54 --- fixed


(Reporter: kanru, Assigned: ted)




(1 file)

I was trying to load some minidumps for bug 1325258 but found some minidumps cannot be loaded in VS successfully.

For example the minidump from bp-447ac0fc-664a-4227-9242-4e2fe2170117 is good but the minidump from bp-1a85eb8a-3c57-45ba-b1f6-e1f772170117 is broken.

I'm not sure if this is a symbol server issue or we changed the way we generate symbols for symbol server. If this is not the correct component, please correct me.
Flags: needinfo?(ted)
This is a regression from bug 1329320, sorry. I did test it locally but apparently not thoroughly enough. I added a bunch of tests to the Rust crate and I believe it works interoperably with the Microsoft implementation now:

I'll update the in-tree binaries which should fix this.
Assignee: nobody → ted
Blocks: 1329320
Flags: needinfo?(ted)
Comment on attachment 8830020 [details]
bug 1332147 - update makecab to a newer version.
Attachment #8830020 - Flags: review?(mshal) → review+
We should uplift this to Aurora as well since bug 1329320 was merged there.
Pushed by
update makecab to a newer version. r=mshal
Closed: 6 years ago
Resolution: --- → FIXED
I picked an arbitrary Nightly crash:

Downloaded the minidump, opened it in MSVC, and verified that I was able to debug it without issues.
Component: Symbols → Breakpad Integration
Product: Socorro → Toolkit
Target Milestone: --- → mozilla54
Version: unspecified → Trunk
Comment on attachment 8830020 [details]
bug 1332147 - update makecab to a newer version.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1329320
[User impact if declined]: Debugging minidumps from crash-stats on Windows is more difficult for developers. They may need to download the matching build instead of the debugger automatically fetching the files from the symbol server.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: The code in question here is only used for compressing PDB and EXE/DLL files for upload to the symbol server, which are only provided for use by developers using a debugger. That's already somewhat broken on Aurora so it can't get much more broken, and it has zero impact on the product or crash reporting.
[String changes made/needed]: None
Attachment #8830020 - Flags: approval-mozilla-aurora?
Comment on attachment 8830020 [details]
bug 1332147 - update makecab to a newer version.

This can help debug crash. Aurora53+.
Attachment #8830020 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.