[Webkit-unassigned] [Bug 45391] QtWebKit asserts when selecting elided text.

[bugzilla-daemon at webkit.org]

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


Ademar Reis <ademar.reis at openbossa.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ademar.reis at openbossa.org




--- Comment #21 from Ademar Reis <ademar.reis at openbossa.org>  2010-12-09 11:25:18 PST ---
(In reply to comment #20)
> (In reply to comment #19)
> > Shouldn't this be cherry picked into qtwebkit-2.1 ? There is a downstream report of a crash that seems to have been caused by this bug. See https://bugs.kde.org/show_bug.cgi?id=259272.
> 
> I would not cherry pick targeting 2.1 for an assert. It is late in the process, only important changes should go in.
> Qt scripts to build webkit always build in release.

I believe a fix for a crash that happens by openging a web page is worth including in the release (or in a later minor update)... It can even be classified as a security vuln (at minimum it's a DoS).

The problem I have is that I can't reproduce the crash with the supplied example:

Inside Font::selectionRectForText(), codePath(run) returns Complex and thus Font::selectionRectForSimpleText() (where the problem is) never gets called.

Anybody familiar with the code could tell what should I do to actually call Font::selectionRectForSimpleText()?

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