bugGnash - The GNU Flash player - Bugs: bug #34314, Segfault in Button::getMovieInfo

 
 

bug #34314: Segfault in Button::getMovieInfo

Submitted by:  Sandro Santilli <strk>
Submitted on:  Sat 17 Sep 2011 01:11:32 PM UTC  
 
Category: coreSeverity: 5 - Blocker
Release: masterStatus: Fixed
Privacy: PublicAssigned to: Benjamin Wolsey <bwy>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sun 18 Sep 2011 09:27:56 AM UTC, comment #4:

Thanks, fix confirmed. This is the way blocker bugs should be handled !

Sandro Santilli <strk>
Project Member
Sun 18 Sep 2011 08:19:07 AM UTC, comment #3:

Was a null pointer dereference due to my accidentally using a different container.

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Sat 17 Sep 2011 05:46:03 PM UTC, comment #2:

If the backtrace is correct this is on the iteration through Button state characters, so it seems that one is either null or invalid; both appear to break a class invariant. I'll try to look at it more soon.

(It would be interesting to test with 0.8.9 too because I'm not sure I did touch that particular line after all).

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Sat 17 Sep 2011 05:26:00 PM UTC, comment #1:

Yes, almost certainly related to my commit.

Benjamin Wolsey <bwy>
Project MemberIn charge of this item.
Sat 17 Sep 2011 01:11:32 PM UTC, original submission:

commit: 2cba2b (pre-ffmpeg-hell)
movie: http://www.ilsole24ore.com/Creativita_Flash/Notizie/2011/09/italia-provincia-ricchi/italia-province-ricchi.swf

Clicking on File->Properties triggers a segfault

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff5a1a9c3 in boost::_mfi::mf2<tree<std::pair<std::string, std::string>, std::allocator<tree_node_<std::pair<std::string, std::string> > > >::pre_order_iterator, gnash::DisplayObject, tree<std::pair<std::string, std::string>, std::allocator<tree_node_<std::pair<std::string, std::string> > > >&, tree<std::pair<std::string, std::string>, std::allocator<tree_node_<std::pair<std::string, std::string> > > >::pre_order_iterator>::operator() (this=<value optimized out>,
tr=<value optimized out>, it=...)
at /usr/include/boost/bind/mem_fn_template.hpp:274
274 BOOST_MEM_FN_RETURN (p->*f_)(a1, a2);
.....
#4 gnash::Button::getMovieInfo (this=<value optimized out>,
tr=<value optimized out>, it=...)
at /usr/src/gnash/gnash-head/libcore/Button.cpp:968

Ben, is this for you ?

Sandro Santilli <strk>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bwy (Posted a comment)
  • -unavailable- added by strk (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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 18 Sep 2011 09:27:56 AM UTCstrkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Sun 18 Sep 2011 08:19:07 AM UTCbwyStatusNone=>Ready For Test
    Sat 17 Sep 2011 05:26:00 PM UTCbwyAssigned toNone=>bwy
    Sat 17 Sep 2011 01:12:06 PM UTCstrkSummaryFile->Properties segfaults=>Segfault in Button::getMovieInfo

    Back to the top


    Powered by Savane 3.1-cleanup1