Closed Bug 10558 Opened 25 years ago Closed 25 years ago

extra copy in nsStdURL::ReconstructPath()

Categories

(Core :: Networking, defect, P3)

Product:
Core
Component:
Networking
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: gagan)

Details

(Whiteboard: waiting for feedback) 

nsStdURL::ReconstructPath() allocates a copy of a bit of char* data, does an
operation, strdups it and then frees the original copy.

Can we get rid of the second copy and just use the first? (tempPath)
Target Milestone: M9
Is this Necko build specific?  What build?  Did you regress against the M9 trunk
build?  bruce, we need better into please.
Status: NEW → ASSIGNED
Yes Jan this is Necko only. Anything marked with Necko component is Necko-only.

Bruce: I know about this and as you might read in the code comments this was a
temporary hack. I have also sent out a mail in this regard which will explain
why I was doing this. Here is the link-

news://news.mozilla.org/379D373B.31495C60%40netscape.com
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: waiting for feedback
gagzilla, you took this out, then? can I mark verified?
Yes this is definitely fixed.
Status: RESOLVED → VERIFIED
Bulk move of all Necko (to be deleted component) bugs to new Networking

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