Closed Bug 276424 Opened 21 years ago Closed 21 years ago

[nsJARProtocolHandler::NewURI] leaks nsIURI

Categories

(Core :: Networking: File, defect)

Product:
Core
Component:
Networking: File
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: Mook, Assigned: darin.moz)

References

Details

Attachments

(1 file)

Trivial fix.
1.06 KB, patch
Details | Diff | Splinter Review
http://lxr.mozilla.org/mozilla/source/modules/libjar/nsJARProtocolHandler.cpp#173 If aBaseURI->Resolve fails, a nsURI is leaked. Patch in a second.
Attached patch Trivial fix. — — Splinter Review
I hope I got this right :)
Attachment #169874 - Flags: review?
Attachment #169874 - Flags: review? → review?(bsmedberg)
I think you did, but bug 278262 makes this patch unnecessary. I'll leave it in my queue until that bugs gets checked in.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Fixed by checkin for bug 278262
Status: NEW → RESOLVED
Closed: 21 years ago
Depends on: 278262
Resolution: --- → FIXED
Attachment #169874 - Flags: review?(benjamin)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: