FYI - would BeOS and Netware folks please review this commit? It appears to me that neither actually uses fork-for-exec, and both certainly fail to call the apr_pool_cleanup_for_exec() mandatory hook if they do use that approach.
Bill [EMAIL PROTECTED] wrote:
Author: wrowe Date: Mon Oct 15 15:07:32 2007 New Revision: 584943 URL: http://svn.apache.org/viewvc?rev=584943&view=rev Log: OS2 fails to fork-for-exec, and therefore doesn't use the apr_pool_cleanup_for_exec hook, which means it's leaking like a sieve. This alternate implementation of the function is sub-optimal because parallel threads can be setting up handles to be inherited, and this just isn't healthy. The best fix is to create all handles uninherited, and only toggle the std streams to be inherited in apr_proc_create(). I'm happy to work with any OS2 folk to make that happen.
