Open Bug 1866496 Opened 1 year ago Updated 1 year ago

Add support for the HANDLE / file descriptor lists

Categories

(Socorro :: Webapp, task, P3)

Tracking

(Not tracked)

People

(Reporter: gsvelto, Unassigned)

References

Details

When bug 1864000 lands Firefox minidumps will contain a list of handles (on Windows) and file descriptors (on Linux) which we'd like to be able to inspect in Socorro. This data contains PII so we'd like only a small part of it to be surfaced in the public part of the report, and the rest to be gated only to people with the appropriate access.

The way the new data is stored in the stackwalker's JSON output is described here and this data is available starting with version 0.19.1 of minidump-stackwalk (so we'll need to update to that version to actually have it in the output).

For the public part we'd like to add one line to the "Details" tab, holding the number of handles. That's just the number of entries in the "handles" array. If it's not available, or if it's zero the line should be omitted. For Windows minidumps this should be called "HANDLE count" and for all other OSes "File descriptors count".

For people with data access it would be nice to have an additional tab holding the list of handles. This would be a table with three columns:

  • The first holding the handle value (that is handles[i].handle), this is just a number
  • The second one holding the type name if it's not empty (handles[i].type_name)
  • The third one holding the object name if it's not empty (handles[i].object_name)

The third column is really the one that contains PII; because it will contain paths within the user filesystem. I don't know if it's feasible but I guess we could make this tab public, provided that the data in the third column is not visible to people without protected data access.

Depends on: 1864000

We've updated the stackwalker to v0.21.1, so this work is unblocked now. I don't know when we're going to get to it.

Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.