Closed Bug 986076 Opened 11 years ago Closed 11 years ago

Handle JS_NewStringCopyZ failure in ProxyAutoConfig.cpp

Categories

(Core :: Networking, defect)

Product:

Component:

Type:

defect

Priority:

Not set

Severity:

normal

Tracking

()

Status:

RESOLVED FIXED

Milestone:

mozilla31

People

(Reporter: poiru, Assigned: poiru)

References

Details

Attachments

(1 file)

Handle JS_NewStringCopyZ failure in ProxyAutoConfig.cpp 11 years ago Biru [:poiru] 5.64 KB, patch	mcmanus : review+	Details \| Diff \| Splinter Review

Assignee

Description

•

11 years ago

See bug 845478 comment 119.

Assignee

Comment 1

•

11 years ago

Attached patch Handle JS_NewStringCopyZ failure in ProxyAutoConfig.cpp — Details — Splinter Review

Try push: https://tbpl.mozilla.org/?tree=Try&rev=54b4e5a2e9fe

Attachment #8395886 - Flags: review?(mcmanus)

Patrick McManus [:mcmanus]

Comment 2

•

11 years ago

Comment on attachment 8395886 [details] [diff] [review] Handle JS_NewStringCopyZ failure in ProxyAutoConfig.cpp Review of attachment 8395886 [details] [diff] [review]: ----------------------------------------------------------------- i appreciate the updates

Attachment #8395886 - Flags: review?(mcmanus) → review+

Assignee

Updated

•

11 years ago

Keywords: checkin-needed

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Comment 3

•

11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/513ffcfe773d

Keywords: checkin-needed

Ryan VanderMeulen [:RyanVM]

Comment 4

•

11 years ago

https://hg.mozilla.org/mozilla-central/rev/513ffcfe773d

Status: ASSIGNED → RESOLVED

Closed: 11 years ago

Resolution: --- → FIXED

Target Milestone: --- → mozilla31

You need to log in before you can comment on or make changes to this bug.