C / C++ / MFC

void ReadAllSections(const CString &sFileName, CStringArray &arrsSections)
{
    static const DWORD k_dwBufferInc = 10;
    DWORD dwBufferSize = 0, dwReturnedLength;
    CString sBuffer; // using a CString as a buffer to simplify memory housekeeping

    // try to read the section names; increase buffer and retry if too small
    do
    {
        dwBufferSize += k_dwBufferInc;
        dwReturnedLength = GetPrivateProfileSectionNames(sBuffer.GetBuffer(dwBufferSize - 1), dwBufferSize, sFileName);
    }
    while (dwReturnedLength == dwBufferSize - 2);

    arrsSections.RemoveAll();

    if (dwReturnedLength == 0)
    {
        // dwLastErr is 0 if the file or directory don't exist, or if it exists and contains no sections.
        // Sometimes, on a Vista Home PC, 8 is returned, although the file exists and contains sections.
        DWORD dwLastErr = GetLastError();
        AfxDebugBreak();
        return;
    }

    // split buffer into section names (I know this part could me more sophisticated)
    LPCTSTR pszCurrent = sBuffer;
    while (*pszCurrent != _T('\0'))
    {
        arrsSections.Add(pszCurrent);
        pszCurrent += _tcslen(pszCurrent) + 1;
    }
}

hans.sch wrote:

About ReleaseBuffer() - omitting it, in my understanding, may confuse CString operations but should not make an API call fail should it?