FIX: _AfxFullPath() Uses lstrcpy Incorrectly If Errors Occur

The information in this article applies to:

• The Microsoft Foundation Classes (MFC), included with: Microsoft Visual C++ for Windows, version 1.0

SYMPTOMS

When an application uses either CWinApp::OpenDocumentFile() or CFile::GetStatus() and passes a filename that includes an invalid path, _AfxFullPath() recognizes the problem but doesn't return the path specified.

CAUSE

In the file AUXDATA.CPP, the following lines are executed within _AfxFullPath() when the path to the file cannot be parsed correctly:

   if (OpenFile(lpszFileIn, &of, OF_PARSE) != HFILE_ERROR)
   {
       ...
   }
   else
   {
    TRACE1("Warning: could not parse the path %Fs\n", lpszFileIn);
       lstrcpy(lpszPathOut, lpszPathOut);  // take it literally
    AnsiUpper(lpszPathOut); // paths in uppercase just to be sure
       return FALSE;
   }

RESOLUTION

To solve this problem, replace the second argument of lstrcpy with lpszFileIn in AUXDATA.CPP as shown below:

   lstrcpy(lpszPathOut, lpszPathIn);

STATUS

Microsoft has confirmed this to be a problem with version 2.0 of the Microsoft Foundation Class Libraries. This problem was corrected in the Microsoft Foundation Classes version 2.5.

NOTE: This problem does not occur in the Microsoft Foundation Classes version 2.0 included with Visual C++, 32-bit Edition, version 1.0.