[Webkit-unassigned] [Bug 19853] REGRESSION (r34838): Crash when visiting http://www.thewebsiteisdown.com/salesguy.html

[bugzilla-daemon at webkit.org]

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


cwzwarich at uwaterloo.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |DUPLICATE




------- Comment #6 from cwzwarich at uwaterloo.ca  2008-07-03 06:25 PDT -------
As Alexey mentioned, the problem is _NPN_Invoke not being able to handle a 0
return value from Machine::execute(). The particular 0 in question is coming
from

    Register* r = slideRegisterWindowForCall(exec, newCodeBlock,
&m_registerFile, m_registerFile.base(), callFrame, argv, argc, *exception);
    if (*exception) {
        m_registerFile.shrink(oldSize);
        return 0;
    }

However, the call to slideRegisterWindowForCall() does not set an exception, so
the exception must be sitting around from before. Indeed, it was set by
_NPN_SetException(). The fix for bug 19736 is to simply remove the body of
_NPN_SetException() because it is completely incorrect and not doing anything
useful at the moment. The question of how to properly handle exceptions from
the NPAPI is somewhat thorny, because of compatibility issues.

I'm marking this as a duplicate.

*** This bug has been marked as a duplicate of 19736 ***


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