[Webkit-unassigned] [Bug 29908] New: Crash when UTF8 is not supported by platform TextCodec

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=29908

           Summary: Crash when UTF8 is not supported by platform TextCodec
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Text
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: yong.li at torchmobile.com
                CC: staikos at kde.org


UTF8Encoding() returns a static TextEncoding object. But if "UTF-8" is not
registered by platform TextCodec, the encoding object will contain a null
"m_name" and newTextCodec() will get a null "factory" function.

There's already an ASSERT(factory.function) in newTextCodec(), but at that
point, the encoding name is null. It's not so clear that the crash is due to
missing UTF-8 support. ASSERT(factory.function) is supposed to detect the
problem that the platform TextCodec forgets to register a factory for an
encoding it has registered.

So we think adding an ASSERT to UTF8Encoding() can be helpful.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.