Bug 50840 - similar bugs to apachebug45464
Summary: similar bugs to apachebug45464
Status: RESOLVED INVALID
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_dav (show other bugs)
Version: 2.2.17
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-27 21:52 UTC by Linhai Song
Modified: 2011-03-03 14:02 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Linhai Song 2011-02-27 21:52:32 UTC
After reading bug report of apachebug45464 (https://issues.apache.org/bugzilla/show_bug.cgi?id=45464). I think if apr_stat is called with APR_FINFO_NORM as the third parameter on windows platform, similar performance problem will happen. So I check the latest httpd code carefully, and find several places where apr_stat is called with APR_FINFO_NORM as the third parameter. 

I list them below, should they be patched? and should they be patched in the same way as apachebug45464.

===================================================================
./modules/dav/fs/repos.c:462:    rv = apr_stat(&src_finfo, src, APR_FINFO_NORM, p);

====================================================================
./modules/dav/fs/repos.c:483:    rv = apr_stat(&dst_state_finfo, dst, APR_FINFO_NORM, p);

====================================================================
./modules/dav/fs/repos.c:796:                  APR_FINFO_NORM, ctx->pool);
rv = apr_stat(&parent_ctx->finfo, parent_ctx->pathname,
                 APR_FINFO_NORM, ctx->pool);

=====================================================================
./os/unix/unixd.c:253:                  APR_FINFO_NORM, ptemp)) != APR_SUCCESS) {

if ((apr_stat(&wrapper, SUEXEC_BIN,
                 APR_FINFO_NORM, ptemp)) != APR_SUCCESS) {
       return;

====================================================================
./srclib/apr/test/sendfile.c:149:    rv = apr_stat(&finfo, fname, APR_FINFO_NORM, p);

====================================================================
./srclib/apr/test/testdup.c:52:    rv = apr_stat(&finfo, FILEPATH "testdup.file", APR_FINFO_NORM, p);

====================================================================
./srclib/apr/test/testdup.c:97:    rv = apr_stat(&finfo, FILEPATH "testdup.readwrite.file", APR_FINFO_NORM, p);

====================================================================
./srclib/apr/test/testfileinfo.c:136:    rv = apr_stat(&finfo, FILENAME, APR_FINFO_NORM, p);
====================================================================;
./srclib/apr/test/testfileinfo.c:166:    rv = apr_stat(&stat_finfo, FILENAME, APR_FINFO_NORM, p);
====================================================================
./srclib/apr/test/testlfs.c:136:                       apr_stat(&finfo, TESTFN, APR_FINFO_NORM, p));
Comment 1 Linhai Song 2011-03-03 14:02:26 UTC
I made some mistakes on my report. 

Don't bother to look at this bug report.

Sorry for that.