Closed Bug 1271887 Opened 7 years ago Closed 7 years ago

Crash signature containing IPC::ParamTraits<T> should not be aggregated to the generic T type

Categories

(Socorro :: Backend, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kanru, Assigned: adrian)

Details

Attachments

(1 file)

IPC::ParamTraits<T> is implemented for each type T that handles serialization and deserialization of the type. They usually have very different implementation and call stacks. So for example IPC::ParamTraits<mozilla::net::NetAddr>::Write is different with IPC::ParamTraits<mozilla::gfx::gfxPoint> and shouldn't be aggregated to the same IPC::ParamTraits<T>::Write crashes.
How urgent is this?
Assignee: nobody → adrian
Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/d7843027fb6335ffa63da05647b2d29506a84532
Fixes bug 1271887 - Do not normalize template types for IPC::ParamTraits. (#3343)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
This is not urgent but will be nice to have it. Thanks for acting so fast! Will this landing affects the future signatures only?
Flags: needinfo?(adrian)
Yes, unless we reprocess affected crash reports. It's not easy for us to do that at the moment, but we are working on automating the process in the near future. 

If reprocessing those old crash reports is important to your work, please file a new bug asking for reprocessing, ideally with a SuperSearch of the concerned reports. :)
Flags: needinfo?(adrian)
You need to log in before you can comment on or make changes to this bug.