bugGnash - The GNU Flash player - Bugs: bug #19174, unchecked use of Range2d

 
 

bug #19174: unchecked use of Range2d

Submitted by:  Sandro Santilli <strk>
Submitted on:  Wed 28 Feb 2007 10:37:38 PM UTC  
 
Category: coreSeverity: 4 - Important
Release: NoneStatus: 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.

 

Wed 28 Feb 2007 10:54:47 PM UTC, comment #1:

Found the problem, it seems that the Range covers a sub-pixel
area, and when scaled to pixel has a dimensional collapse.
The problem was actually in Range2d<> implementation insisting
on min values being < then max values.
I think for them being equal is fine. Committed this change.

Sandro Santilli <strk>
Project MemberIn charge of this item.
Wed 28 Feb 2007 10:37:38 PM UTC, original submission:

Sounds like AGG renderer is calling Range2d<T>::scale
against a NULL range ?
Movie is: http://www.ferryhalim.com/orisinal/sunshine/all.swf

lt-gnash: /home/strk/src/gnash/gnash-head/libgeometry/Range2d.h:487: gnash::geometry::Range2d<T>& gnash::geometry::Range2d<T>::scale(float, float) [with T = float]: Assertion `_xmin < _xmax' failed.

Program received signal SIGABRT, Aborted.
[Switching to Thread -1227278656 (LWP 28120)]
0xffffe410 in __kernel_vsyscall ()
(gdb) bt
#0 0xffffe410 in __kernel_vsyscall ()
#1 0xb6e739a1 in raise () from /lib/tls/i686/cmov/libc.so.6
#2 0xb6e752b9 in abort () from /lib/tls/i686/cmov/libc.so.6
#3 0xb6e6cf51 in __assert_fail () from /lib/tls/i686/cmov/libc.so.6
#4 0xb7a47f55 in gnash::geometry::Range2d<float>::scale (this=0xbf9855bc, xfactor=0.0500000007, yfactor=1) at Range2d.h:487
#5 0xb7a4802d in gnash::geometry::Range2d<float>::scaleX (this=0xbf9855bc, factor=0.0500000007) at Range2d.h:430
#6 0xb7a48058 in gnash::rect::scale_x (this=0xbf9855bc, factor=0.0500000007) at rect.h:166
#7 0xb7a61c52 in gnash::render_handler_agg<agg::pixfmt_alpha_blend_rgb<agg::blender_rgb_pre<agg::rgba8, agg::order_rgb>, agg::row_accessor<unsigned char> > >::select_clipbounds (this=0x8203e00, def=0x8222800, mat=@0xbf985778) at /home/strk/src/gnash/gnash-head/backend/render_handler_agg.cpp:828
#8 0xb7a681a8 in gnash::render_handler_agg<agg::pixfmt_alpha_blend_rgb<agg::blender_rgb_pre<agg::rgba8, agg::order_rgb>, agg::row_accessor<unsigned char> > >::draw_shape_character (this=0x8203e00, def=0x8222800, mat=@0xbf985778, cx=@0xbf985758, fill_styles=@0x8222814, line_styles=@0x8222820)
at /home/strk/src/gnash/gnash-head/backend/render_handler_agg.cpp:892
#9 0xb7a3dd41 in gnash::render_handler::draw_shape_character (this=0x8203e00, def=0x8222800, inst=0x823a1a8) at render_handler.h:392

Sandro Santilli <strk>
Project MemberIn charge of this item.

 

(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 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 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 28 Feb 2007 10:54:47 PM UTCstrkCategoryrender-agg=>core
      StatusNone=>Fixed
      Assigned toudog=>strk
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1