I'm sorry... this patch dir not come through to [EMAIL PROTECTED] for me today
(although I watched for it...) but it's simply WRONG.



At 07:04 PM 12/17/2002, =?UTF-8?B?QnJhbmtvIMSMaWJlag==?= wrote:
>>--- apr/file_io/unix/dir.c     15 Dec 2002 05:17:51 -0000      1.69
>>+++ apr/file_io/unix/dir.c     17 Dec 2002 00:49:35 -0000
>>@@ -218,10 +218,10 @@
>>         return ret;
>>     }
>> 
>>-#ifdef DIRENT_INODE
>>+#ifndef DIRENT_INODE
>>     wanted &= ~APR_FINFO_INODE;
>> #endif

Old logic; if we have an INODE from dirent, we don't care that we 
want an INODE from stat() because we already have the INODE.

New Logic: if we don't have an INODE, we won't ask for an INODE
from stat().

I'm sorry, but that's just broken.
'
Please revert and (re)post the original description of the problem.

If you pass APR_FINFO_TYPE | APR_FINFO_INDOE | APR_FINFO_NAME
that is *ALL* you are promised... we do NOT stat for info you don't ask for.

Bill

Reply via email to