[Webkit-unassigned] [Bug 55704] QuickTime plugin should opt in to a 32-bit non-executable heap

[bugzilla-daemon at webkit.org]

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





--- Comment #8 from Damian Kaleta <dkaleta at apple.com>  2011-03-04 09:41:00 PST ---
I misunderstood this bug, let me clarify the ChangeLog and make changes to the patch.

(In reply to comment #7)
> This patch is confused. The ChangeLog states that Silverlight and QuickTime should opt *in* to a 32-bit NX heap, which means their heap should not be executable. But then it opts them *out* by making pluginNeedsExecutableHeap() claim that those two plugins need an executable heap — i.e. that they cannot run with NX. It also opts all other plugins into running with an NX heap.
> 
> Assuming you're really trying to say "QuickTime and Silverlight should run with a non-executable heap (opt-in), and other plugins should have an executable one (opt-out)", you need to invert your true/false returns in pluginNeedsExecutableHeap.

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