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

 
 

bug #34314: Segfault in Button::getMovieInfo

Submitter:  Sandro Santilli <strk>
Submitted:  Sat 17 Sep 2011 01:11:32 PM UTC
   
 
Category:  core Severity:  5 - Blocker
Release:  master Status:  Fixed
Privacy:  Public Assigned to:  bwy
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group Member
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>
Group Member
Sat 17 Sep 2011 05:26:00 PM UTC, comment #1: 

Yes, almost certainly related to my commit.

Benjamin Wolsey <bwy>
Group Member
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>
Group Member

 

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

    Date Changed by Updated Field Previous Value => Replaced by
    2011-09-18 strk StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2011-09-18 bwy StatusNone Ready For Test
    2011-09-17 bwy Assigned toNone bwy
    2011-09-17 strk SummaryFile-&gt;Properties segfaults Segfault in Button::getMovieInfo

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code