bugGnash - The GNU Flash player - Bugs: bug #32965, gnash crashes on loading flash...

 
 

bug #32965: gnash crashes on loading flash page with konqueror

Submitter:  Christoph Thielecke <crissi>
Submitted:  Thu 31 Mar 2011 12:12:58 PM UTC
   
 
Category:  plugin Severity:  4 - Important
Release:  master Status:  Fixed
Privacy:  Public Assigned to:  bjacques
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 12 Sep 2011 05:43:33 PM UTC, comment #10: 

Please file another bug for youtube. This one was for the crash.
Thanks.

Sandro Santilli <strk>
Group Member
Mon 12 Sep 2011 01:25:44 PM UTC, comment #9: 

it does not crash but videos on youtube (e.g. interaction with server) does not work.

Christoph Thielecke <crissi>
Mon 12 Sep 2011 12:18:05 PM UTC, comment #8: 

crissi: is this fixed for you ?

Sandro Santilli <strk>
Group Member
Thu 31 Mar 2011 04:35:09 PM UTC, comment #7: 

Please don't start a flame war in the bug tracker. Splitting hairs over how we differ on defining Blocker is a waste of time. To me a Blocker keeps Gnash for working at all, whereas Important is for things that Gnash still runs mostly, but has crashes sometimes. We should fix all important and blocker bugs for a release. Making everything a blocker basically ignores the Important value completely.

Rob Savoye <rsavoye>
Group administrator
Thu 31 Mar 2011 04:31:01 PM UTC, comment #6: 

We haven't fixed all Important bugs for 0.8.9
But we did fix all the Blockers.
And by we I mean everyone but you...

Please don't make it hard to keep gnash on an acceptable quality level.


Sandro Santilli <strk>
Group Member
Thu 31 Mar 2011 04:21:10 PM UTC, comment #5: 

It is up to the maintainer to decide the bug severity as we've discussed before. Making most of your filed bugs blockers makes that value meaningless. All important and blocker bugs should be fixed by the next release as a matter of process.

I believe konqueror is using the NPAPI emulation available under WebKit, which may not support GetURL the same. It'd be worth checking.

Rob Savoye <rsavoye>
Group administrator
Thu 31 Mar 2011 04:09:31 PM UTC, comment #4: 

Is blocking next release of course, not the past one.

Are you sure it takes a scriptable-aware npapi version
to have NPN_GetURL ?

Sandro Santilli <strk>
Group Member
Thu 31 Mar 2011 04:06:19 PM UTC, comment #3: 

Older browsers don't support getURL anyway, and a blocker on something already released is useless. As this is in master too, we should check the value of _scriptObject to avoid crashes.

Rob Savoye <rsavoye>
Group administrator
Thu 31 Mar 2011 03:57:14 PM UTC, comment #2: 

Please also try http://strk.keybit.net/tmp/getURL.swf as I belive there's another unchecked access of _scriptObject to be taken care of.

Sandro Santilli <strk>
Group Member
Thu 31 Mar 2011 12:34:55 PM UTC, comment #1: 

Looks like _scriptObject is null, condition which isn't checked in ::startProc, nor in processPlayerRequest.

Given _scriptObject is only set when NPNFuncs.version >= 14 I wonder if this means older browser won't have any support for GETURL either.

Sandro Santilli <strk>
Group Member
Thu 31 Mar 2011 12:12:58 PM UTC, original submission:  

If I try to load a page with flash inside it crashes here on konqueror (3.5.9).

I installed dbg packages and attach the backtrace.

Christoph Thielecke <crissi>

 

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

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by bjacques (Updated the item)
  • -email is unavailable- added by rsavoye (Updated the item)
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by crissi (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.

     

    Follow 14 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-09-12 strk StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2011-04-10 bjacques StatusNone Ready For Test
        Assigned toNone bjacques
    2011-03-31 crissi Attached File- Added gnash_crash2.txt, #23068
    2011-03-31 rsavoye Severity5 - Blocker 4 - Important
    2011-03-31 strk Severity4 - Important 5 - Blocker
    2011-03-31 rsavoye Release0.8.9 master
    2011-03-31 rsavoye Severity5 - Blocker 4 - Important
    2011-03-31 crissi Attached File- Added gnash_crash_fix.patch, #23066
    2011-03-31 strk Categoryrelease 0.8.9 plugin
        Severity3 - Normal 5 - Blocker
        ReleaseNone 0.8.9
    2011-03-31 crissi Attached File- Added gnash_crash.txt, #23064

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code