Open
Bug 977778
Opened 11 years ago
Updated 4 years ago
Allow users to request get-minidump-instructions report on-demand
Categories
(Socorro :: Processor, task, P3)
Tracking
(Not tracked)
NEW
People
(Reporter: benjamin, Unassigned)
References
Details
Ted wrote a tool called get-minidump-instructions which will print out and disassemble the memory surrounding the instruction pointer. The JS team would like a way to run this tool on minidumps in an on-demand way.
This is similar to bug 818069 in that it's a report that we don't want to run on every minidump by default for storage and processor-time reasons.
Unlike bug 818069, however, this report can potentially contain PII and should only be requestable or viewable by people who have the "download raw minidumps" privilege.
Comment 1•11 years ago
|
||
Do you think this is PII? It's just the memory around the instruction pointer, which should just be code.
Comment 2•11 years ago
|
||
Comment 3•10 years ago
|
||
"we should treat it as PII for now" -bsmedberg
Comment 4•10 years ago
|
||
On second thought this makes sense--if you jump into random memory you will get whatever memory is around where things blow up.
So two quick thoughts:
1) We could just spit the raw bytes out in the stackwalker JSON. This would mean we'd have to eat that extra storage for each dump, and it'd also mean we'd have to do the disassembly in JS (emscripten objdump or something?)
2) If we do this server-side we need objdump installed to do the actual disassembly (get-minidump-instructions just shells out).
Comment 5•10 years ago
|
||
Here's a basic first step that adds this tool to the Socorro build:
https://github.com/mozilla/socorro/pull/2544
Comment 6•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/10f28335c7950dfe1930d0999804b4d013f8d79d
bug 977778 - add get-minidump-instructions binary to Socorro
https://github.com/mozilla/socorro/commit/fe044f6a701809bd097e5199306d42b17a625792
Merge pull request #2544 from luser/get-minidump-instructions
bug 977778 - add get-minidump-instructions binary to Socorro
Comment 7•9 years ago
|
||
The get-minidump-instructions tool takes the same command-line arguments as the stackwalker binary, but also requires one extra argument: --disassemble (if that's a problem I can change the code to make that the default).
A local invocation looks like this:
get-minidump-instructions --disassemble --symbols-url=http://symbols.mozilla.org/ --symbols-cache=/tmp/symbols ~/dumps/bug1202817-85785551-d76b-43e2-b245-77b312150908.dmp
attachment 8658472 [details] contains the output, it's currently just plain text, although if we made it JSON we could do nicer formatting on it (and better links to the source etc), but that's a nice-to-have followup.
As for a key to save it under in the JSON dump, doesn't particularly matter to me, maybe "crash_disassembly" ?
We will need to make sure whatever is going to run this tool (the processor?) has `objdump` installed, I don't know how hard that is.
It would also be great if we could run this on-demand on specific crashes, since that's the most pressing use case (bug 818069 wants that too).
Comment 8•7 years ago
|
||
Moving this into the Processor component so it's not lost in the shuffle.
Component: General → Processor
Comment 9•5 years ago
|
||
I've had several conversations about this sort of thing over the last few years. Making it a P3 to get to eventually.
Priority: -- → P3
You need to log in
before you can comment on or make changes to this bug.
Description
•