patchGnash - The GNU Flash player - Patches: patch #8916, Patch to make libgnashplugin...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #8916: Patch to make libgnashplugin return real JavaScript object from ExternalInterface callback

Submitted by:  Nutchanon Wetchasit <nachanon>
Submitted on:  Mon 22 Feb 2016 01:57:20 PM UTC  
 
Category:  plugin Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  Sandro Santilli <strk> Open/Closed:  Closed

Thu 03 Mar 2016 09:33:17 AM UTC, comment #4: 

Pushed as commit 765adb072dc4b8407f1936a60c3f2af4aeff2046

Sandro Santilli <strk>
Project MemberIn charge of this item.
Thu 03 Mar 2016 08:44:46 AM UTC, comment #3: 

I guess we can just commit this and try it on the field...

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 23 Feb 2016 08:24:59 AM UTC, comment #2: 

FWIW, compiler reported this:

/usr/src/gnash/gnash-head/plugin/npapi/plugin.cpp:930:14: warning: variable 'invokeResult' set but not used [-Wunused-but-set-variable]
         bool invokeResult = false;
              ^

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 23 Feb 2016 08:24:21 AM UTC, comment #1: 

I'm not experienced with NPAPI either, so can't help much with it.
Do you want to take a look, Rob ?

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 22 Feb 2016 01:57:20 PM UTC, original submission:  

Current NPAPI libgnashplugin code returns a custom-allocated `NPObject`
to browser when it receives Object-typed or Array-typed return value from
player-side ExternalInterface callback.

Doing this resulting in object's members not being set correctly
(`NPN_SetProperty()` returned false), JavaScript seen the object
as type `function`, and the plugin container crashed after it
received such `NPObject` instance.

The attached `0001_make-libgnashplugin-return-real-js-object.patch` does following things:

  • Drop the custom `NPObject` allocation in `ExternalInterface::parseXML()`.
  • Make `ExternalInterface::parseXML()` call container's `window.Object()` method to construct a JavaScript-authentic `Object`.
  • Change `NPP nppinstance` field in `GnashPluginScriptObject` to public, allowing `ExternalInterface::parseXML()` to access it.
  • Change various functions which call `ExternalInterface::parseXML()`, to make them pass `GnashPluginScriptObject` along.
  • Implement `NPN_GetValue()` and `NPN_Invoke()` stub in standalone tester.
  • Comment out existing `<object>` parsing test as `GnashPluginScriptObject` and container's `window` object are not available in testing environment.

This fixes the crash and single-depth Object member unmarshalling in
JavaScript-calls-Flash ExternalInterface return value, partially fixing
bug #32411, while allowing unrelated tests in plugin's standalone tester
to continue working.

This patch needs review since I'm new to Netscape Plugin API framework
and its NPRuntime extension, and I'm not sure if there's a nicer way to
implement this (especially the way that `NPP` instance got passed around).

Gnash: 0.8.11dev (patched against git c12c3cf 22-Feb-2016) NPAPI
Browser: Iceweasel 10.0.12 (debian)
System: Debian GNU/Linux 7.0 Wheezy i386

Nutchanon Wetchasit <nachanon>

 

Attached Files
file #36423:  0001_make-libgnashplugin-return-real-js-object.patch added by nachanon (20KiB - text/x-diff - Patch for making NPAPI libgnashplugin return real JavaScript Object for ExternalInterface object return value)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by nachanon (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-03-03 strk StatusIn Progress => Done
        Open/ClosedOpen => Closed
    2016-03-03 strk StatusNone => In Progress
        Assigned torsavoye => strk
    2016-02-23 strk Assigned toNone => rsavoye
    2016-02-22 nachanon Attached File- => Added 0001_make-libgnashplugin-return-real-js-object.patch, #36423

    Back to the top


    Powered by Savane 3.5