C ++-調用std :: vector的.clear（）時訪問沖突

Question

我正在Visual Studio C ++ MFC應用程序中實現XLSX電子表格閱讀器，並且多次執行時遇到訪問沖突錯誤：

First-chance exception at 0x7720e39e in VCT.exe: 0xC0000005: Access violation reading location 0x02bdddab.
Unhandled exception at 0x7720e39e in VCT.exe: 0xC0000005: Access violation reading location 0x02bdddab.
The program '[1756] VCT.exe: Native' has exited with code -1073741819 (0xc0000005).

奇怪的是，根據我的代碼的其余部分，這可能在函數被調用兩次，三次或更多次之后發生。這使我認為這與計時有關，但是我只有一個線程運行。 我的另一個假設（更現實）是它是未定義的行為。 這使得調試特別困難。 讓我感到困惑的是，為什么在多次調用該函數后會發生這種訪問沖突。

添加到問題：

每次單擊按鈕時，我都會調用函數getVectorAllIP 。 幾次單擊（調用getVectorAllIP ）后，第一次調用mm_XLSXReader.xlsxGetCellOffset ，我收到訪問沖突錯誤。

vector<CString> CVCTDlg::getVectorAllIP(string ipName){

    CString CSIP1;

    vector<CString> VCSIPAddresses;

    XLSXReader mm_XLSXReader;
    mm_XLSXReader.reloadFile();
    mm_XLSXReader.setFilePath(XLSX_FILE_PATH);

    for(int i = 0; i < m_vectorStrHostname.size(); i++)
    {
        CSIP1="";
        for(int iOffset = 0; iOffset < 4; iOffset++)
        {

            CSIP1.Append(mm_XLSXReader.xlsxGetCellOffset(ipName, i, iOffset).c_str());
            if(iOffset != 3)
            {
                CSIP1.Append(".");
            }
        }

        if(CSIP1 != "...")
        {

            VCSIPAddresses.push_back(CSIP1);

        }else{
            VCSIPAddresses.push_back("");
        }

    }

    return VCSIPAddresses;

}

在xlsxGetCellOffset內發生訪問沖突錯誤readSheetXml 。

string XLSXReader::xlsxGetCellOffset(string columnName, int id, int offset)
{
    string contentToReturn;
    id++;

    if(!m_bFileInMemory)
    {
        if(openm_XLSXReader())
        {
            readSharedStringsXml();
            readSheetXml();

            closem_XLSXReaders();

            m_bFileInMemory = true;
        }
    }

    for(int i = 0; i < m_header.size(); i++)
    {
        if(m_header.at(i) == columnName && m_header.size() > i + offset)
        {
            if(m_sheetContent.size() > id)
            {
                if(m_sheetContent.at(id).size() > i)
                {
                    contentToReturn = m_sheetContent.at(id).at(i+offset);
                }
            }
        }
    }

    return contentToReturn;
}

訪問沖突在最后的清除序列期間發生。 特別是在columnContent.clear() 。 如果刪除columnContent.clear()它將出現在下一行tParameterColumn.clear() 。

void XLSXReader::readSheetXml()
{
    if(m_m_XLSXReader)
    {
        int error = unzLocateFile(m_m_XLSXReader, SHEET_NAME, 0);
        if(error == UNZ_OK)
        {
            error = unzOpenCurrentFile(m_m_XLSXReader);
            if(error == UNZ_OK)
            {
                int readingStatus = 0;
                char readBuffer[BUFFERSIZE];
                string file;
                int indexValue;
                //Reading File
                do
                {
                    readingStatus = unzReadCurrentFile(m_m_XLSXReader, readBuffer, BUFFERSIZE);
                    file.append(readBuffer, readingStatus);
                }while(readingStatus > 0);


                //Sort Data
                vector<string> rowContent;
                rowContent = findXmlTagsContent(file, "row");

                unsigned int iHdrSize;
                m_header.clear();

                vector<string> columnContent;
                vector<string> tParameterColumn;
                vector<string> rParameterColumn;
                vector<string> tmpRow;
                for(int i = 0 ; i < rowContent.size(); i++)
                {

                    columnContent=findXmlTagsContent( rowContent.at(i), "c"); 
                    rParameterColumn=findXmlParameterInTag(rowContent.at(i), "c", "r");
                    tParameterColumn=findXmlParameterInTag(rowContent.at(i), "c", "t");
                    if(i==0){
                        iHdrSize = columnContent.size();
                    }

                    //Should be equal
                    if(columnContent.size() == tParameterColumn.size())
                    {
                        unsigned int iFilledColumns = 0;
                        for(int j = 0 ; j < columnContent.size(); j++)
                        {               

                            int columnNumber = 0;

                            if(!rParameterColumn.at(j).empty())
                            {
                                columnNumber = columnLetter2Int(rParameterColumn.at(j));
                            }

                            vector<string> value;
                            value = findXmlTagsContent(columnContent.at(j), "v");

                            if(value.size()>1){
                                value.clear();
                                value.push_back("");
                            }

                            //Header Reading
                            if( i == 0)
                            {
                                //Fill empty spaces in excel sheet with ""
                                for(int a = 1; a < columnNumber-iFilledColumns; a++)
                                {
                                    m_header.push_back("");
                                }
                                iFilledColumns=m_header.size();

                                //link to sharedString
                                if(tParameterColumn.at(j) == "s")
                                {
                                    indexValue = atoi(value.at(0).c_str());
                                    string tmpStr = m_sharedString.at(indexValue);
                                    m_header.push_back(tmpStr);
                                }
                                //Value
                                else
                                {   
                                    m_header.push_back(value.at(0));
                                }
                            }
                            // Content Reading
                            else
                            {
                                ////Fill empty spaces in excel sheet with ""
                                for(int a = 1; a < columnNumber-iFilledColumns; a++)
                                {
                                    tmpRow.push_back("");
                                }
                                iFilledColumns=tmpRow.size();
                                //link to sharedString

                                if(tParameterColumn.at(j) == "s")
                                {
                                    indexValue = atoi(value.at(0).c_str());
                                    tmpRow.push_back(m_sharedString.at(indexValue));
                                }
                                //Value
                                else
                                {
                                    if(value.size() != 0)
                                    {
                                        tmpRow.push_back(value.at(value.size()-1));
                                    }
                                    else
                                    {
                                        tmpRow.push_back("");
                                    }
                                }

                            }
                            iFilledColumns++;
                        }
                        for(int k=0;k<iHdrSize-iFilledColumns;k++){
                            tmpRow.push_back("");
                        }
                        m_sheetContent.push_back(tmpRow);
                        tmpRow.clear();
                        columnContent.clear();
                        tParameterColumn.clear();
                        rParameterColumn.clear();

                    }
                }       
            }
        }
    }
}

而僅供參考， m_m_XLSXReader被實例化到一個呼叫openm_XLSXReader稱為內xlsxGetCellOffset 。 這里供您參考：

bool XLSXReader::openm_XLSXReader()
{
    //Uncompress .xlsx
    m_m_XLSXReader = unzOpen(m_strXLSXPath.c_str());

    if(m_m_XLSXReader){
        return true;
    }
    return false;
}

希望有人指出一些明顯的錯誤，因為我開始質疑我的理智:)謝謝。

Answer 1

這個循環：

do
{
    readingStatus = unzReadCurrentFile(m_m_XLSXReader, readBuffer, BUFFERSIZE);
    file.append(readBuffer, readingStatus);
} while (readingStatus > 0);

將最后一次讀取的塊追加兩次，很可能產生無效的XML。 我不知道您用來讀取XML的庫（除了這個問題之外，找不到任何對findXmlTagsContent引用），以及它的彈性如何，但是我懷疑它在喂入垃圾時可能表現不佳。 另外，您不檢查任何可能的錯誤...

底線是：嘗試像這樣讀取文件：

while ((readingStatus = unzReadCurrentFile(m_m_XLSXReader, readBuffer, BUFFERSIZE)) > 0)
    file.append(readBuffer, readingStatus);

另外，如果返回值為負（錯誤代碼），您將怎么辦？

Answer 2

好的，謝謝大家的幫助和建議。

弗拉德（Vlad），您的建議確實讓我以更合乎邏輯的方式思考，然后我發現了與實際XLSX讀數完全無關的問題。

簡單的解釋是，當我單擊啟動XLSX閱讀器的按鈕時，我在此之前修改了Windows注冊表。 在遞歸刪除某些注冊表項的過程中，會發生某種形式的內存損壞，只有當我遇到訪問沖突時才會反映出來。 我修復了我的注冊表代碼，現在問題已解決。

如果有人對遞歸刪除注冊表項的實際問題感興趣，請繼續閱讀...

我使用代碼遞歸刪除注冊表項及其子項：

刪除帶有子鍵的鍵

#include <windows.h>
#include <stdio.h>
#include <strsafe.h>

//*************************************************************
//
//  RegDelnodeRecurse()
//
//  Purpose:    Deletes a registry key and all its subkeys / values.
//
//  Parameters: hKeyRoot    -   Root key
//              lpSubKey    -   SubKey to delete
//
//  Return:     TRUE if successful.
//              FALSE if an error occurs.
//
//*************************************************************

BOOL RegDelnodeRecurse (HKEY hKeyRoot, LPTSTR lpSubKey)
{
    LPTSTR lpEnd;
    LONG lResult;
    DWORD dwSize;
    TCHAR szName[MAX_PATH];
    HKEY hKey;
    FILETIME ftWrite;

    // First, see if we can delete the key without having
    // to recurse.

    lResult = RegDeleteKey(hKeyRoot, lpSubKey);

    if (lResult == ERROR_SUCCESS) 
        return TRUE;

    lResult = RegOpenKeyEx (hKeyRoot, lpSubKey, 0, KEY_READ, &hKey);

    if (lResult != ERROR_SUCCESS) 
    {
        if (lResult == ERROR_FILE_NOT_FOUND) {
            printf("Key not found.\n");
            return TRUE;
        } 
        else {
            printf("Error opening key.\n");
            return FALSE;
        }
    }

    // Check for an ending slash and add one if it is missing.

    lpEnd = lpSubKey + lstrlen(lpSubKey);

    if (*(lpEnd - 1) != TEXT('\\')) 
    {
        *lpEnd =  TEXT('\\');
        lpEnd++;
        *lpEnd =  TEXT('\0');
    }

    // Enumerate the keys

    dwSize = MAX_PATH;
    lResult = RegEnumKeyEx(hKey, 0, szName, &dwSize, NULL,
                           NULL, NULL, &ftWrite);

    if (lResult == ERROR_SUCCESS) 
    {
        do {

            StringCchCopy (lpEnd, MAX_PATH*2, szName);

            if (!RegDelnodeRecurse(hKeyRoot, lpSubKey)) {
                break;
            }

            dwSize = MAX_PATH;

            lResult = RegEnumKeyEx(hKey, 0, szName, &dwSize, NULL,
                                   NULL, NULL, &ftWrite);

        } while (lResult == ERROR_SUCCESS);
    }

    lpEnd--;
    *lpEnd = TEXT('\0');

    RegCloseKey (hKey);

    // Try again to delete the key.

    lResult = RegDeleteKey(hKeyRoot, lpSubKey);

    if (lResult == ERROR_SUCCESS) 
        return TRUE;

    return FALSE;
}

//*************************************************************
//
//  RegDelnode()
//
//  Purpose:    Deletes a registry key and all its subkeys / values.
//
//  Parameters: hKeyRoot    -   Root key
//              lpSubKey    -   SubKey to delete
//
//  Return:     TRUE if successful.
//              FALSE if an error occurs.
//
//*************************************************************

BOOL RegDelnode (HKEY hKeyRoot, LPTSTR lpSubKey)
{
    TCHAR szDelKey[MAX_PATH*2];

    StringCchCopy (szDelKey, MAX_PATH*2, lpSubKey);
    return RegDelnodeRecurse(hKeyRoot, szDelKey);

}

void __cdecl main()
{
   BOOL bSuccess;

   bSuccess = RegDelnode(HKEY_CURRENT_USER, TEXT("Software\\TestDir"));

   if(bSuccess)
      printf("Success!\n");
   else printf("Failure.\n");
}

如果我嘗試刪除的鍵中包含值，那么一切都會按預期進行。 如果我有一個帶有子鍵的鍵，它們將被刪除，但是我開始遇到上述問題。

因此，例如刪除了該文件卻沒有給我帶來痛苦，

[KeyName1]
--> SOME_DWORD
--> SOME_STRING

這確實使我很痛苦，

[KeyName2]
--> SOME_DWORD
--> SOME_STRING
--> [SubKeyName]
-----> SOME_DWORD
-----> SOME_STRING