bugGnash - The GNU Flash player - Bugs: bug #32044, plugin crashes

 
 

bug #32044: plugin crashes

Submitted by:  Gabriele Giacone <gg0>
Submitted on:  Tue 04 Jan 2011 12:21:12 AM UTC  
 
Category: pluginSeverity: 5 - Blocker
Release: trunkStatus: Fixed
Privacy: PublicAssigned to: Sandro Santilli <strk>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Sat 12 Feb 2011 04:22:08 PM UTC, comment #12:

Thanks for testing.
Pushed to master 9b39d4778c6a5781fce26294122293b5a416b90e

Sandro Santilli <strk>
Project MemberIn charge of this item.
Sat 12 Feb 2011 04:17:39 PM UTC, comment #11:

Your patch works great. No more crashes.

Gabriele Giacone <gg0>
Project Member
Sat 12 Feb 2011 04:05:36 PM UTC, comment #10:

Checking some returns from std::string::find makes it a much more stable run for me. Patch here:

http://strk.keybit.net/tmp/0002-Check-return-from-std-string-find.patch

Sandro Santilli <strk>
Project MemberIn charge of this item.
Fri 11 Feb 2011 12:53:38 PM UTC, comment #9:

With returnfalse, I can play all movies, no crashes.

Gabriele Giacone <gg0>
Project Member
Fri 11 Feb 2011 08:34:43 AM UTC, comment #8:

Can you try adding a 'return false' right at the start of processPlayerRequest in plugin/npapi/plugin.cpp and see how youtube works then ?

bool
nsPluginInstance::processPlayerRequest(gchar* buf, gsize linelen)
{
return false;

Sandro Santilli <strk>
Project MemberIn charge of this item.
Fri 11 Feb 2011 03:16:14 AM UTC, comment #7:

Attached backtrace from build 88edb2b9d2916 on "release" branch.

(file #22648)

Gabriele Giacone <gg0>
Project Member
Thu 10 Feb 2011 04:58:19 PM UTC, comment #6:

Built from master 783adeb0c607c6

Debian amd64 iceweasel 3.6.13

Gabriele Giacone <gg0>
Project Member
Thu 10 Feb 2011 04:48:58 PM UTC, comment #5:

valgrind report attached

(file #22646)

Gabriele Giacone <gg0>
Project Member
Wed 05 Jan 2011 07:39:50 PM UTC, comment #4:

in my case it happens just with plugin, when it start it crash, with standalone.

-X- <maximi89>
Wed 05 Jan 2011 04:23:42 PM UTC, comment #3:

Only the plugin, or is it reproducible in the standalone? Does it happen for particular SWFs, or does it happen for any?

It's an error in the mark stage of the GC, most likely trying to mark an object that doesn't exist. It could also be due to an earlier memory corruption, so a valgrind report would be helpful.

Benjamin Wolsey <bwy>
Project Member
Wed 05 Jan 2011 04:40:32 AM UTC, comment #2:

This happens at start right?

in the case f propietary, it fails when it's in use, so not related at all...

-X- <maximi89>
Wed 05 Jan 2011 04:14:50 AM UTC, comment #1:

this happens with propietary plugin too, probably something related bith both projects?
when it crash you, it is in hard CPU usage?

-X- <maximi89>
Tue 04 Jan 2011 12:21:12 AM UTC, original submission:

Randomly happens.
Attached bt and full bt.

Gabriele Giacone <gg0>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #22646:  valgrind.log added by gg0 (381KiB - text/x-log)
file #22342:  bt.txt added by gg0 (43KiB - text/plain)
file #22343:  bt_full.txt added by gg0 (80KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by strk (Updated the item)
  • -unavailable- added by bwy (Posted a comment)
  • -unavailable- added by maximi89 (Posted a comment)
  • -unavailable- added by gg0 (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 12 Feb 2011 04:22:08 PM UTCstrkStatusNone=>Fixed
      Assigned toNone=>strk
      Open/ClosedOpen=>Closed
    Fri 11 Feb 2011 03:16:14 AM UTCgg0Attached File-=>Added bt_88edb2b9_release_branch.log, #22648
    Thu 10 Feb 2011 04:54:06 PM UTCstrkSeverity3 - Normal=>5 - Blocker
    Thu 10 Feb 2011 04:48:58 PM UTCgg0Attached File-=>Added valgrind.log, #22646
    Tue 04 Jan 2011 12:21:12 AM UTCgg0Attached File-=>Added bt_full.txt, #22343
      Attached File-=>Added bt.txt, #22342

    Back to the top


    Powered by Savane 3.1-cleanup1