[ob/fix] libc/stdio/fread.c: Include malloc.h

Jeff Johnston jjohnstn@redhat.com
Mon Sep 27 15:36:00 GMT 2004


Corinna Vinschen wrote:
> Hi,
> 
> the latest patch to fread.c uses FREEUB.  This is defined in
> libc/stdio/local.h and calls _free_r().  _free_r doesn't exist on
> Cygwin and is therefore defined as free in stdlib.h as well as in
> malloc.h.  Unfortunately, none of these files is included in
> fread.c, nor in local.h so this patch breaks Cygwin.
> 
> I've applied the obvious fix below.  Perhaps, long-term it would
> be better to include stdlib.h or malloc.h in local.h.
> 
> 
> Corinna
> 
> 	* libc/stdio/fread.c (fread): Include <malloc.h>.
> 
> Index: libc/stdio/fread.c
> ===================================================================
> RCS file: /cvs/src/src/newlib/libc/stdio/fread.c,v
> retrieving revision 1.6
> diff -p -u -r1.6 fread.c
> --- libc/stdio/fread.c  22 Sep 2004 21:22:07 -0000      1.6
> +++ libc/stdio/fread.c  24 Sep 2004 09:07:08 -0000
> @@ -58,6 +58,7 @@ Supporting OS subroutines required: <<cl
>  #include <_ansi.h>
>  #include <stdio.h>
>  #include <string.h>
> +#include <malloc.h>
>  #include "local.h"
>  
>  #ifdef __SCLE
>

My bad.  I actually had the fix in "local.h" but neglected to check it in with 
the fread.c fix.

-- Jeff J.



More information about the Newlib mailing list