RE: [Devel] [BUG]: ftsystem.c: free( block );

[Turner David]

Hello,

> 
> Hello, dear FreeType team,
> at debuging of new Y graphic system I think I found bug at ftsystem.c 
> files(s):
> may be routine:
>   FT_CALLBACK_DEF( void )
>   ft_free( FT_Memory  memory,
>            void*      block )
>   {
>     FT_UNUSED( memory );
> 
>     free( block );
>   }
> should be replaced by:
>   FT_CALLBACK_DEF( void )
>   ft_free( FT_Memory  memory,
>            void*      block )
>   {
>     FT_UNUSED( memory );
> 
>     if ( block ) free( block );
>   }

This is not a bug. "ft_free" is a function called from FT_Free
which itself already checks for NULL pointers.

> Y scanned my /usr/share/fonts/default/ghostscript directory where 
> FT_Done_Face() called ft_free().
> Excuse me I cannot include the "nasty" fonts but I have incidently 
> erased them. I am not sure but I think the rpm 
> ftp://216.254.0.38/linux/redhat/updates/7.3/en/os/i386/ghostsc
> ript-6.52-9.5.i386.rpm 
> contain the same "nasty" fonts.
> regards,
> --vlindos
> 
> 

The problem might be different. Did you try with the latest CVS version.
We fixed a bug that caused double-frees and other ugliness lately.

Regards,

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

--
This message and any attachments (the "message") is intended solely for the
addressees and is confidential. If you receive this message in error, please
delete it and immediately notify the sender.
Any use not in accordance with its purpose, any dissemination or disclosure,
either whole or partial, is prohibited except formal approval.
The E-Mail transmission can not guarantee the integrity of this message.
CANAL+TECHNOLOGIES will not therefore be liable for the message if modified.