Re: Access Denied error on calling CFile::GetStatus() function

From:

"Tom Serface" <tom.nospam@camaswood.com>

Newsgroups:

microsoft.public.vc.mfc

Date:

Fri, 23 Mar 2007 19:48:35 -0700

Message-ID:

<18888C88-5F88-4353-B501-C811E5E3FDB7@microsoft.com>

Hmmm, I didn't know your problem was with an exception. I also did the same
sort of thing a year or so ago with this function for the same issue. I
just call this instead of the CFile version. Sorry, I didn't know you had
the same problem and I didn't know they "didn't" fix it for 2005 since I
don't use that function in CFile any longer. I'm surprised though. We've
been griping about it failing on invalid dates since around version 5 :o)
and the fix is pretty easy.

Tom

BOOL GetStatus(LPCTSTR lpszFileName, CFileStatus& rStatus)
{
     ASSERT( lpszFileName != NULL );

     if ( lpszFileName == NULL )
          return FALSE;

     if ( lstrlen(lpszFileName) >= _MAX_PATH ) {
          ASSERT(FALSE); // MFC requires paths with length < _MAX_PATH
          return FALSE;
     }

     // attempt to fully qualify path first
     if (!AfxFullPath(rStatus.m_szFullName, lpszFileName)) {
          rStatus.m_szFullName[0] = _T('\0');
          return FALSE;
     }

     WIN32_FIND_DATA findFileData;
     HANDLE hFind = FindFirstFile(lpszFileName, &findFileData);
     if (hFind == INVALID_HANDLE_VALUE)
          return FALSE;
     VERIFY(FindClose(hFind));

     // strip attribute of NORMAL bit, our API doesn't have a "normal" bit.
     rStatus.m_attribute = (BYTE)
          (findFileData.dwFileAttributes & ~FILE_ATTRIBUTE_NORMAL);

     // get just the low DWORD of the file size
    // ASSERT(findFileData.nFileSizeHigh == 0);
     rStatus.m_size = (LONG)findFileData.nFileSizeLow;

     // convert times as appropriate
     try {
          rStatus.m_ctime = CTime(findFileData.ftCreationTime);
     }
     catch(...) {
          rStatus.m_ctime = CTime::GetCurrentTime();
     }

     try {
          rStatus.m_atime = CTime(findFileData.ftLastAccessTime);
     }
     catch(...) {
          rStatus.m_atime = rStatus.m_ctime;
     }

     try {
          rStatus.m_mtime = CTime(findFileData.ftLastWriteTime);
     }
     catch(...) {
          rStatus.m_atime = rStatus.m_ctime;
     }

     if (rStatus.m_ctime.GetTime() == 0)
          rStatus.m_ctime = rStatus.m_mtime;

     if (rStatus.m_atime.GetTime() == 0)
          rStatus.m_atime = rStatus.m_mtime;

     return TRUE;
}

"David Connet" <dcon@agilityrecordbook.com> wrote in message
news:IOedndi8LqQs_pnbnZ2dnUVZ_g6dnZ2d@comcast.com...

=?Utf-8?B?U2FyYXRo?= <Sarath@discussions.microsoft.com> wrote in
news:D3F8708C-BCA3-4C21-AA1A-9B97A4D40060@microsoft.com:

It was a typo that I passed & of FileStatus Object. In the actualy
code I've written it properly.

The last error was set to 0.

I shalll try with the code you specified. But the CFile::GetStatus
working properly with the directories in of local drive but not
working fine with the network file paths.

Ah joy. You have to replace CFile::GetStatus. Often, network/cd
drives/etc have bad dates. When GetStatus reads the info, the underlaying
CTime throws. VC8 does NOT fix this. I've seen cases where
CTime::IsValidFILETIME returns true, but CTime(filetime) throws.

What I did was to copy the GetStatus code from the MFC source. Then
around each status time field do the following (this is based on the VC8
source):
if (CTime::IsValidFILETIME(findFileData.ftCreationTime))
{
try
{
rStatus.m_ctime = CTime(findFileData.ftCreationTime);
}
catch (COleException* pe)
{
pe->Delete();
rStatus.m_ctime = CTime();
}
}
else
{
rStatus.m_ctime = CTime();
}

Dave Connet