[Webkit-unassigned] [Bug 118505] Leak: TextCodecICU::registerCodecs is leaking
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 9 09:13:40 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=118505
Alexey Proskuryakov <ap at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #206319|review? |review-
Flag| |
--- Comment #2 from Alexey Proskuryakov <ap at webkit.org> 2013-07-09 09:15:40 PST ---
(From update of attachment 206319)
I think that the leak may be real, but the fix is not right.
Generally, these strings are permanently stored in textCodecMap, and not leaked. One case where a leak may occur is when the same webStandardName is registered multiple times - when this happens, map value is thrown away.
However, I don't think that it's correct to remove fastStrDup. The result of ucnv_getCanonicalName is not documented to remain useful indefinitely - for example, they may be reusing a static buffer. So, we need our own copy of the string.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list