Closed Bug 663229 Opened 8 years ago Closed 8 years ago

e10s HTTP: initialize prnetaddrs correctly in child

Categories

(Core :: Networking: HTTP, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla7

People

(Reporter: jduell.mcbugs, Assigned: jduell.mcbugs)

Details

Attachments

(1 file)

Spin-off of bug 661158.  Just minor codefix to move some initializers that should now be in HttpBaseChannel, not nsHttpChannel.
Copying jdm's +r.

Includes breakpoint to cause parent to crash (intentionally) if we see it trying to pass in a Unix domain socket enum over IPDL.
Attachment #538347 - Flags: review+
Keywords: checkin-needed
Whiteboard: [inbound → [inbound]
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla7
Assignee: nobody → jduell.mcbugs
Comment on attachment 538347 [details] [diff] [review]
Approved patch copied from bug 661158

So, this patch does not actually fix the topcrash that is tracked by bug 661158. In fact, it arguably makes the crash strictly worse, since it changes a content process crash into a browser crash. On the other hand, this change should make us crash at the point when we start sending bad data over the wire, and this is good because Jason and I can't figure out why this would ever happen. Obviously it is, and it's happening fairly frequently, so this kind of data would be valuable to us.
Attachment #538347 - Flags: approval-mozilla-aurora?
Comment on attachment 538347 [details] [diff] [review]
Approved patch copied from bug 661158

> this change should make us crash at the point when we start sending bad 
> data over the wire, and this is good 

+1
We didn't have a mobile rep at today's triage but desktop has no concerns here so I'm delegating the yes/no to which ever of the Mobile drivers gets here first.
Comment on attachment 538347 [details] [diff] [review]
Approved patch copied from bug 661158

since this doesn't actually fix anything I don't think we want to take this. If my understanding is incorrect, please explain and re-nom
Attachment #538347 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
You need to log in before you can comment on or make changes to this bug.