Iain Wade wrote:
New summary:
dir.patch:
1/ add apr_dir_put_ex, adding dirname and flags
2/ refactor apr_dir_open to use apr_dir_put_ex
3/ add apr_dir_name_get
4/ add apr_dir_pool_get
file.patch:
1/ add apr_os_file_put_ex, changing behavior to honour
APR_FILE_NOCLEANUP set/unset.
2/ refactor apr_file_open to use apr_dir_put_ex
OK. So the _ex variants register a cleanup unless you pass
APR_(FILE|DIR)_NOCLEANUP flag - so both behaviours are possible.
I have tested the privsep patches with changes to use these new APIs and
they work as expected.
There is no longer a need to delve into the internals of the apr_dir_t
struct to get a working apr_dir_t from a apr_os_dir_t or to access non
public APIs to get files cleaned up when apr_file_close is called on an
apr_file_t created with apr_os_file_put_ex.
I've also tested these patches with a standard httpd-trunk exercising
the normal path apr_dir_open/apr_file_open.
