RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

2003-11-26 Joel Sherrill <joel@OARcorp.com>

Browse Source 
PR 524/filesystem
	* src/newlibc.c: Address 1K buffer leak from incomplete cleanup of
	newlib reentrancy handler.
This commit is contained in:
Joel Sherrill
2003-11-26 17:53:30 +00:00
parent 690861c3c6
commit c9a4da01c6
2 changed files with 34 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cpukit/libcsupport/ChangeLog
View File

@@ -1,3 +1,9 @@
2003-11-26 Joel Sherrill <joel@OARcorp.com>
PR 524/filesystem
* src/newlibc.c: Address 1K buffer leak from incomplete cleanup of
newlib reentrancy handler.
2003-11-26 Joel Sherrill <joel@OARcorp.com> 2003-11-26 Joel Sherrill <joel@OARcorp.com>
PR 523/filesystem PR 523/filesystem

28
cpukit/libcsupport/src/newlibc.c
View File

@@ -42,6 +42,8 @@
#include <stdio.h> #include <stdio.h>
int _fwalk(struct _reent *ptr, int (*function) (FILE *) );
int libc_reentrant; /* do we think we are reentrant? */ int libc_reentrant; /* do we think we are reentrant? */
struct _reent libc_global_reent; struct _reent libc_global_reent;
@@ -196,6 +198,26 @@ rtems_extension libc_begin_hook(rtems_tcb *current_task)
* *
*/ */
int newlib_free_buffers(
FILE *fp
)
{
switch ( fileno(fp) ) {
case 0:
case 1:
case 2:
if (fp->_flags & __SMBF) {
free( fp->_bf._base );
fp->_flags &= ~__SMBF;
fp->_bf._base = fp->_p = (unsigned char *) NULL;
}
break;
default:
fclose(fp);
}
return 0;
}
rtems_extension libc_delete_hook( rtems_extension libc_delete_hook(
rtems_tcb *current_task, rtems_tcb *current_task,
rtems_tcb *deleted_task rtems_tcb *deleted_task
@@ -215,8 +237,14 @@ rtems_extension libc_delete_hook(
/* if (ptr) */ /* if (ptr) */
if (ptr && ptr != &libc_global_reent) { if (ptr && ptr != &libc_global_reent) {
/*
_wrapup_reent(ptr); _wrapup_reent(ptr);
_reclaim_reent(ptr); _reclaim_reent(ptr);
*/
/*
* Just in case there are some buffers lying around.
*/
_fwalk(ptr, newlib_free_buffers);
#if REENT_MALLOCED #if REENT_MALLOCED
free(ptr); free(ptr);
#else #else