From 8860f103a57220cfe005275ad542cde65adedc4d Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Thu, 20 Feb 2003 22:09:50 +0000 Subject: Patch against RTEMS-ss-20030128 Author: Till Straumann 2003/2/11 This patch addresses the following issues: 2003-02-20 Till Straumann PR 349/bsps * src/mount.c, src/privateenv.c: - mount() bugfix: allocated 'loc' node must be released if mount fails due to missing node_type_h - silence compiler warning about improper pointer type Index: cpukit/libcsupport/src/mount.c =================================================================== RCS file: /afs/slac/g/spear/cvsrep/rtems/src-20030128/cpukit/libcsupport/src/mount.c,v retrieving revision 1.1.1.2 diff -c -r1.1.1.2 mount.c *** cpukit/libcsupport/src/mount.c 29 Jan 2003 22:57:19 -0000 1.1.1.2 --- cpukit/libcsupport/src/mount.c 7 Feb 2003 20:47:49 -0000 *************** *** 142,147 **** --- 142,149 ---- mount_point, RTEMS_LIBIO_PERMS_RWX, &loc, TRUE ) == -1 ) goto cleanup_and_bail; + loc_to_free = &loc; + /* * Test for node_type_h */ *************** *** 155,161 **** * Test to see if it is a directory */ - loc_to_free = &loc; if ( loc.ops->node_type_h( &loc ) != RTEMS_FILESYSTEM_DIRECTORY ) { errno = ENOTDIR; goto cleanup_and_bail; --- 157,162 ---- Index: cpukit/libcsupport/src/privateenv.c =================================================================== RCS file: /afs/slac/g/spear/cvsrep/rtems/src-20030128/cpukit/libcsupport/src/privateenv.c,v retrieving revision 1.1.1.3 diff -c -r1.1.1.3 privateenv.c *** cpukit/libcsupport/src/privateenv.c 29 Jan 2003 22:57:20 -0000 1.1.1.3 --- cpukit/libcsupport/src/privateenv.c 30 Jan 2003 18:01:40 -0000 *************** *** 65,71 **** tmp->refcnt = 1; #endif ! sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,free_user_env); if (sc != RTEMS_SUCCESSFUL) { /* don't use free_user_env because the pathlocs are * not initialized yet --- 65,71 ---- tmp->refcnt = 1; #endif ! sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,(void(*)(void *))free_user_env); if (sc != RTEMS_SUCCESSFUL) { /* don't use free_user_env because the pathlocs are * not initialized yet --- cpukit/libcsupport/ChangeLog | 66 +++++++++++++++++++++++++++++++++++++ cpukit/libcsupport/src/mount.c | 3 +- cpukit/libcsupport/src/privateenv.c | 2 +- 3 files changed, 69 insertions(+), 2 deletions(-) (limited to 'cpukit/libcsupport') diff --git a/cpukit/libcsupport/ChangeLog b/cpukit/libcsupport/ChangeLog index 557656a0c2..28a39ca703 100644 --- a/cpukit/libcsupport/ChangeLog +++ b/cpukit/libcsupport/ChangeLog @@ -1,3 +1,69 @@ +Patch against RTEMS-ss-20030128 +Author: Till Straumann + +2003/2/11 + +This patch addresses the following issues: +2003-02-20 Till Straumann + + PR 349/bsps + * src/mount.c, src/privateenv.c: + - mount() bugfix: allocated 'loc' node must be released if + mount fails due to missing node_type_h + - silence compiler warning about improper pointer type + +Index: cpukit/libcsupport/src/mount.c +=================================================================== +RCS file: /afs/slac/g/spear/cvsrep/rtems/src-20030128/cpukit/libcsupport/src/mount.c,v +retrieving revision 1.1.1.2 +diff -c -r1.1.1.2 mount.c +*** cpukit/libcsupport/src/mount.c 29 Jan 2003 22:57:19 -0000 1.1.1.2 +--- cpukit/libcsupport/src/mount.c 7 Feb 2003 20:47:49 -0000 +*************** +*** 142,147 **** +--- 142,149 ---- + mount_point, RTEMS_LIBIO_PERMS_RWX, &loc, TRUE ) == -1 ) + goto cleanup_and_bail; + ++ loc_to_free = &loc; ++ + /* + * Test for node_type_h + */ +*************** +*** 155,161 **** + * Test to see if it is a directory + */ + +- loc_to_free = &loc; + if ( loc.ops->node_type_h( &loc ) != RTEMS_FILESYSTEM_DIRECTORY ) { + errno = ENOTDIR; + goto cleanup_and_bail; +--- 157,162 ---- +Index: cpukit/libcsupport/src/privateenv.c +=================================================================== +RCS file: /afs/slac/g/spear/cvsrep/rtems/src-20030128/cpukit/libcsupport/src/privateenv.c,v +retrieving revision 1.1.1.3 +diff -c -r1.1.1.3 privateenv.c +*** cpukit/libcsupport/src/privateenv.c 29 Jan 2003 22:57:20 -0000 1.1.1.3 +--- cpukit/libcsupport/src/privateenv.c 30 Jan 2003 18:01:40 -0000 +*************** +*** 65,71 **** + tmp->refcnt = 1; + #endif + +! sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,free_user_env); + if (sc != RTEMS_SUCCESSFUL) { + /* don't use free_user_env because the pathlocs are + * not initialized yet +--- 65,71 ---- + tmp->refcnt = 1; + #endif + +! sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,(void(*)(void *))free_user_env); + if (sc != RTEMS_SUCCESSFUL) { + /* don't use free_user_env because the pathlocs are + * not initialized yet 2003-02-20 Wolfram Wadepohl PR 357/rtems_misc diff --git a/cpukit/libcsupport/src/mount.c b/cpukit/libcsupport/src/mount.c index a137de2ae8..aa682e5532 100644 --- a/cpukit/libcsupport/src/mount.c +++ b/cpukit/libcsupport/src/mount.c @@ -142,6 +142,8 @@ int mount( mount_point, RTEMS_LIBIO_PERMS_RWX, &loc, TRUE ) == -1 ) goto cleanup_and_bail; + loc_to_free = &loc; + /* * Test for node_type_h */ @@ -155,7 +157,6 @@ int mount( * Test to see if it is a directory */ - loc_to_free = &loc; if ( loc.ops->node_type_h( &loc ) != RTEMS_FILESYSTEM_DIRECTORY ) { errno = ENOTDIR; goto cleanup_and_bail; diff --git a/cpukit/libcsupport/src/privateenv.c b/cpukit/libcsupport/src/privateenv.c index d4e5608be1..37d7805eb4 100644 --- a/cpukit/libcsupport/src/privateenv.c +++ b/cpukit/libcsupport/src/privateenv.c @@ -65,7 +65,7 @@ rtems_status_code rtems_libio_set_private_env(void) { tmp->refcnt = 1; #endif - sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,free_user_env); + sc = rtems_task_variable_add(RTEMS_SELF,(void*)&rtems_current_user_env,(void(*)(void *))free_user_env); if (sc != RTEMS_SUCCESSFUL) { /* don't use free_user_env because the pathlocs are * not initialized yet -- cgit v1.2.3