bugGnash - The GNU Flash player - Bugs: bug #50677, Gnash-libgnashplugin communication...

 
 

bug #50677: Gnash-libgnashplugin communication lacks proper escape mechanism

Submitter:  Nutchanon Wetchasit <nachanon>
Submitted:  Thu 30 Mar 2017 07:50:25 AM UTC
   
 
Category:  plugin Severity:  3 - Normal
Release:  master Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 30 Mar 2017 07:50:25 AM UTC, original submission:  

This is a spin-off from bug #46944 (MovieClip-based FSCommand issue).

While I was writing tests for Gnash's FSCommand implementation, I noticed that when Gnash is running as a plug-in, FSCommand call made by the SWF with string parameter full of symbols (especially '<' and '>') will not reach JavaScript FSCommand handler, while ones with normal string parameter will.

Upon inspection, I found that Gnash communication module does not escape '<' and '>' in string content of the message. When '<' is present, the message structure became ambiguous and causes problem with receiver/plugin-side's parser, resulting in discarded message (thus the missing FSCommand call).

This problem is not specific to FSCommand: generic `getURL()` instruction, built-in plugin function like `GetVariable()`, and scripting API like `ExternalInterface` are very likely to be affected too; though these will need additional testing to confirm.

Current automated tests tracking this issue (in FSCommand usage) are:

  • hostcmd_testrunner_v*: (1) (2)
  • hostcmd_htmltest_v*.html: (1) (2)


Gnash: 0.8.11dev (git 8a11e60 8-Mar-2017)
Browser: Iceweasel 10.0.12 (debian)
System: Debian GNU/Linux 7.0 Wheezy i386

Nutchanon Wetchasit <nachanon>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code