Closed Bug 662125 Opened 13 years ago Closed 13 years ago

nsDocShell::ConfirmRepost passes uninitialized PRBool as inout param to nsIPrompt::ConfirmEx

Categories

(Core :: DOM: Navigation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla7

People

(Reporter: khuey, Assigned: khuey)

References

Details

Attachments

(1 file)

I think this also implies that this isn't covered by tests ...
Attached patch PatchSplinter Review
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Attachment #537421 - Flags: review?(bzbarsky)
Comment on attachment 537421 [details] [diff] [review]
Patch

Add a comment indicating the value doesn't actually matter, and r=me.

And yes, this is probably not covered by tests...
Attachment #537421 - Flags: review?(bzbarsky) → review+
http://hg.mozilla.org/mozilla-central/rev/b69d30cc0b24

I'll try to write a test for this as time permits.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: