extra copy in nsStdURL::ReconstructPath()

VERIFIED FIXED in M9

Status

()

Core
Networking
P3
normal
VERIFIED FIXED
19 years ago
9 months ago

People

(Reporter: Bruce Mitchener, Assigned: Gagan)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: waiting for feedback)

(Reporter)

Description

19 years ago
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)

Updated

19 years ago
Target Milestone: M9

Comment 1

19 years ago
Is this Necko build specific?  What build?  Did you regress against the M9 trunk
build?  bruce, we need better into please.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

19 years ago
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
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Updated

19 years ago
Whiteboard: waiting for feedback

Comment 3

19 years ago
gagzilla, you took this out, then? can I mark verified?
(Assignee)

Comment 4

19 years ago
Yes this is definitely fixed.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 5

18 years ago
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.