Functions missing argument lists in Breakpad symbol files




Crash Reporting
2 years ago
2 years ago


(Reporter: calixte, Unassigned)


Firefox Tracking Flags

(Not tracked)



Comment 1

2 years ago
A grep on nsDragService::SetDragIcon shows that in [1] args are presents but not in [2].

Component: General → Breakpad Integration
Product: Socorro → Toolkit
Specifically, the grep output looks like:
/tmp/libxul.sym.1:FUNC 21b2614 1ca 0 nsDragService::SetDragIcon(_GdkDragContext*)
/tmp/libxul.sym.2:FUNC 1be6b20 1a7 0 nsDragService::SetDragIcon

The symbol files conveniently have the changeset information in them, so the regression range here is:

That's a lot of changes, so I built a Mercurial query to look for things that touched the crashreporter, which didn't show me much, so I expanded it to look at browser configs as well:**%22)%20or%20file(%22glob:browser/config/**%22))

...which turns up bug 1250709, which is a very plausible candiate to have broken this!
Summary: Signatures format are different for two similar crashes → Functions missing argument lists in Breakpad symbol files
The first nightly with that change appears to be

downloading the symbol file:

shows that it's missing the parameter list:
FUNC 1bc18e2 1a7 0 nsDragService::SetDragIcon

This shouldn't affect signature generation, since we already strip parameter lists for that, but it will impact readability of crash reports, since it'll be harder to differentiate function overloads.

I'm assuming this is just something different about gcc 4.9's DWARF generation that's breaking dump_syms.
Except bug 1250709 didn't change the compiler for firefox builds, it changed it for rooting hazard builds.
Oh, hah, yeah. That range also includes (bug 1175546) (which wasn't showing up in my hg queries), which is probably it.
You need to log in before you can comment on or make changes to this bug.