Re: [ft-devel] tvtime crashes with freetype cvs on x86-64

[David Turner]

Ronny,

please do not try each commit one after the other, I'll announce it
when I think I'll have something completely fail-safe :-)

Regards,

- David Turner
- The FreeType Project  (www.freetype.org)

On Tue, 05 Sep 2006 15:42:07 +0200, "Ronny V. Vindenes" <address@hidden> said:
> On tir, 2006-09-05 at 13:04 +0200, Ronny V. Vindenes wrote:
> > On man, 2006-09-04 at 20:35 +0200, Ronny V. Vindenes wrote:
> > > On man, 2006-09-04 at 19:00 +0200, Ronny V. Vindenes wrote:
> > > > On man, 2006-09-04 at 18:45 +0200, David Turner wrote:
> > > > > Hello Ronny,
> > > > > 
> > > > > could you please try the current CVS. If this still doesn't work,
> > > > > could you send us the first error reported by Valgrind.
> > > > 
> > > > Current CVS works. Thanks!
> > > > 
> > > 
> > > On closer inspection, I've discovered some rather nasty rendering
> > > errors. On the first line the 'l' is wrong, on the second line the 'c'
> > > is wrong, on the third line 'n' and 'h' are wrong. The errors appear to
> > > depend on entirely on font size. Doing Ctrl-+/- in a browser triggers
> > > rendering errors in different characters and fonts.
> > 
> > Current CVS looks good, thanks for fixing it so quickly David! Attaching
> > a tiny patch to quiet some new compiler warnings.
> 
> You last commit broke tvtime again
> 
> Program received signal SIGSEGV, Segmentation fault.
> 0x00002b651ebdad32 in gray_record_cell (raster=0xa4d3c0)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:389
> 389           if ( cell == NULL || cell->x > x )
> (gdb) bt
> #0  0x00002b651ebdad32 in gray_record_cell (raster=0xa4d3c0)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:389
> #1  0x00002b651ebdb74c in gray_set_cell (raster=0xa4d3c0, ex=63, ey=37)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:457
> #2  0x00002b651ebdb82c in gray_render_scanline (raster=0xa4d3c0, ey=37, 
>     x1=<value optimized out>, y1=232, x2=10778392, y2=0)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:559
> #3  0x00002b651ebdbb1f in gray_render_line (raster=0xa4d3c0,
> to_x=15724, 
>     to_y=7740) at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:737
> #4  0x00002b651ebdbcfe in gray_line_to (to=<value optimized out>,
> raster=0x25)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:1060
> #5  0x00002b651eb9980e in FT_Outline_Decompose (outline=0xa4dac8, 
>     func_interface=0x2b651ee03160, user=0xa4d3c0)
>     at /home/rvv/cvs/freetype2/src/base/ftoutln.c:149
> #6  0x00002b651ebdadec in gray_convert_glyph_inner (raster=0xa4d3c0)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:1573
> #7  0x00002b651ebdb374 in gray_raster_render (raster=0xa4d3c0, 
>     params=<value optimized out>)
>     at /home/rvv/cvs/freetype2/src/smooth/ftgrays.c:1693
> #8  0x00002b651ebda962 in ft_smooth_render_generic (render=0xa4d300, 
>     slot=0x7fff8bf3b490, mode=<value optimized out>, origin=0x0, 
>     required_mode=<value optimized out>, hmul=0, vmul=0)
>     at /home/rvv/cvs/freetype2/src/smooth/ftsmooth.c:198
> #9  0x00002b651ebdaae1 in ft_smooth_render (render=0xa4d3c0, slot=0x40, 
>     mode=4294896448, origin=0xa47718)
>     at /home/rvv/cvs/freetype2/src/smooth/ftsmooth.c:242
> #10 0x00002b651eb9b9cc in FT_Render_Glyph_Internal (library=0x8aec80, 
>     slot=0x7fff8bf3b490, render_mode=FT_RENDER_MODE_NORMAL)
>     at /home/rvv/cvs/freetype2/src/base/ftobjs.c:3269
> #11 0x00002b651eba183d in FT_Glyph_To_Bitmap (the_glyph=0x8ec308, 
>     render_mode=FT_RENDER_MODE_NORMAL, origin=0x0, destroy=1 '\001')
>     at /home/rvv/cvs/freetype2/src/base/ftglyph.c:621
> #12 0x000000000040aa50 in ft_cache_glyph (font=0x8cefc0, wchar=64, 
>     glyph_bbox=0x0) at leetft.c:99
> #13 0x000000000040b32f in ft_font_new (
>     file=0x8aec50 "/usr/share/tvtime/tvtimeSansBold.ttf", 
>     fontsize=<value optimized out>, pixel_aspect=86.625) at leetft.c:171
> #14 0x000000000040c5a8 in osd_font_new (
>     fontfile=0x464af0 "tvtimeSansBold.ttf", fontsize=80,
> pixel_aspect=1.203125)
>     at osdtools.c:60
> #15 0x000000000041266e in tvtime_osd_new (width=<value optimized out>, 
>     height=576, pixel_aspect=1.203125, fieldtime=<value optimized out>, 
>     channel_rgb=4294967040, other_rgb=4294303411) at tvtimeosd.c:248
> #16 0x000000000045ca9d in tvtime_main (rtctimer=0x87e030, read_stdin=0, 
>     realtime=1, argc=<value optimized out>, argv=0x7fff8bf3d698)
>     at tvtime.c:1449
> #17 0x00000000004612d9 in main (argc=1, argv=0x7fff8bf3d698) at
> tvtime.c:2741
> #18 0x0000003a4e020ab4 in __libc_start_main (main=0x4611b0 <main>,
> argc=1, 
>     ubp_av=0x7fff8bf3d698, init=<value optimized out>, 
>     fini=<value optimized out>, rtld_fini=<value optimized out>, 
>     stack_end=0x7fff8bf3d688) from /lib64/libc.so.6
> #19 0x0000000000405049 in _start ()
> 
> from valgrind:
> 
> ==21664== Invalid read of size 4
> ==21664==    at 0x4C75D32: gray_record_cell (ftgrays.c:389)
> ==21664==    by 0x4C7674B: gray_set_cell (ftgrays.c:457)
> ==21664==    by 0x4C7682B: gray_render_scanline (ftgrays.c:559)
> ==21664==    by 0x4C76B1E: gray_render_line (ftgrays.c:737)
> ==21664==    by 0x4C76CFD: gray_line_to (ftgrays.c:1060)
> ==21664==    by 0x4C3480D: FT_Outline_Decompose (ftoutln.c:149)
> ==21664==    by 0x4C75DEB: gray_convert_glyph_inner (ftgrays.c:1573)
> ==21664==    by 0x4C76373: gray_raster_render (ftgrays.c:1693)
> ==21664==    by 0x4C75961: ft_smooth_render_generic (ftsmooth.c:198)
> ==21664==    by 0x4C75AE0: ft_smooth_render (ftsmooth.c:242)
> ==21664==    by 0x4C369CB: FT_Render_Glyph_Internal (ftobjs.c:3269)
> ==21664==    by 0x4C3C83C: FT_Glyph_To_Bitmap (ftglyph.c:621)
> ==21664==  Address 0xFFFEEB40 is not stack'd, malloc'd or (recently)
> free'd
> ==21664== 
> ==21664== Process terminating with default action of signal 11 (SIGSEGV)
> ==21664==  Access not within mapped region at address 0xFFFEEB40
> ==21664==    at 0x4C75D32: gray_record_cell (ftgrays.c:389)
> ==21664==    by 0x4C7674B: gray_set_cell (ftgrays.c:457)
> ==21664==    by 0x4C7682B: gray_render_scanline (ftgrays.c:559)
> ==21664==    by 0x4C76B1E: gray_render_line (ftgrays.c:737)
> ==21664==    by 0x4C76CFD: gray_line_to (ftgrays.c:1060)
> ==21664==    by 0x4C3480D: FT_Outline_Decompose (ftoutln.c:149)
> ==21664==    by 0x4C75DEB: gray_convert_glyph_inner (ftgrays.c:1573)
> ==21664==    by 0x4C76373: gray_raster_render (ftgrays.c:1693)
> ==21664==    by 0x4C75961: ft_smooth_render_generic (ftsmooth.c:198)
> ==21664==    by 0x4C75AE0: ft_smooth_render (ftsmooth.c:242)
> ==21664==    by 0x4C369CB: FT_Render_Glyph_Internal (ftobjs.c:3269)
> ==21664==    by 0x4C3C83C: FT_Glyph_To_Bitmap (ftglyph.c:621)
> 
> -- 
> Ronny V. Vindenes <address@hidden>
> 
- David Turner
- The FreeType Project  (www.freetype.org)