[Devel] Mac fix in CVS? ftmac.c double free bug

[Dan Williams]

Warner,

Can you get this fix from Torrey into CVS in your round of updates too? 
 OpenOffice.org on OS X has been bitten by this issue for a while.

Thanks,
Dan

On Tuesday, April 8, 2003, at 06:49  PM, Torrey Lyons wrote:

> There is a bug in ftmac.c which causes a double free of memory when a 
> face fails to open. A patch to fix this is:
>
> Index: ftmac.c
> ===================================================================
> RCS file: /home/x-cvs/xc/extras/freetype2/src/base/ftmac.c,v
> retrieving revision 1.2
> diff -u -d -b -w -r1.2 ftmac.c
> --- ftmac.c     2003/01/01 22:08:40     1.2
> +++ ftmac.c     2003/04/08 23:43:33
> @@ -545,11 +545,6 @@
>      error = FT_Open_Face( library, &args, face_index, aface );
>      if ( error == FT_Err_Ok )
>        (*aface)->face_flags &= ~FT_FACE_FLAG_EXTERNAL_STREAM;
> -    else
> -    {
> -      FT_Stream_CloseFunc( stream );
> -      FT_FREE( stream );
> -    }
>
>      return error;
>    }
>
> The problem is that FT_Open_Face() already frees the stream with 
> FT_Stream_CloseFunc() and FT_FREE() if there is an error, so we should 
> not be doing it here as well.
>
> --Torrey
>
> _______________________________________________
> Devel mailing list
> address@hidden
> http://www.freetype.org/mailman/listinfo/devel