C ++ bad_alloc是由new引起的?

时间:2013-11-27 19:54:33

标签: c++ arrays bad-alloc

[请注意:我已经阅读了StackOverflow中的现有线程。似乎没有人在我的问题上]

我正在研究Quake 2 MD2格式。在新建一个指针数组后,我似乎得到了一个糟糕的分配。但是,如果我做一些可怕的指针操作,一切都很好。 我想问题是,为什么我会遇到这样的例外?

第一个“MD2Model :: Load”有效。在我获得异常后发布的那个。

基本结构:

struct MD2Header
{
    int nIdentifier;
    int nVersion;

    int nSkinWidth;
    int nSkinHeight;
    int nFrameSize;

    int nNumSkins;
    int nNumVertices;
    int nNumUV;
    int nNumTriangles;
    int nNumCmds;
    int nNumFrames;

    int nOffsetSkins;
    int nOffsetUV;
    int nOffSetTriangles;
    int nOffsetFrames;
    int nOffsetCmds;
    int nOffsetEnd;
};

struct MD2Skin
{
    char szName[64];
};

struct MD2TexCoord
{
    short t;
    short u;
};

struct MD2Triangle
{
    short nVertex[3];
    short tu[3];
};

struct MD2Vertex
{
    float   fVertex[3];
    float   fNormal[3];
};

struct MD2Frame
{
    char szName[16];
    MD2Vertex* pVerts;
};

现在,读取.md2文件的函数:

bool MD2Model::Load( const char* pszName )
{
    FILE* pFile = NULL;

    fopen_s( &pFile, pszName, "rb" );
    if( !pFile )
        return false;

    /* Read Header */
    fread( &m_Header, sizeof(MD2Header), 1, pFile );

    /* Allocate Pointers */
    m_pSkins = new MD2Skin[m_Header.nNumSkins];
    m_pTexCoords = new MD2TexCoord[m_Header.nNumUV];
    m_pTriangles = new MD2Triangle[m_Header.nNumTriangles];
    m_pFrames = new MD2Frame[m_Header.nNumFrames];

    /* Read Skins */
    fseek( pFile, m_Header.nOffsetSkins, SEEK_SET );
    fread( m_pSkins, sizeof(MD2Skin), m_Header.nNumSkins, pFile );

    /* Read Texture Coords */
    fseek( pFile, m_Header.nOffsetUV, SEEK_SET );
    fread( m_pTexCoords, sizeof(MD2TexCoord), m_Header.nNumUV, pFile );

    /* Read Faces */
    fseek( pFile, m_Header.nOffSetTriangles, SEEK_SET );
    fread( m_pTriangles, sizeof(MD2Triangle), m_Header.nNumTriangles, pFile );

    /* Read Animations */
    struct stMD2Vertex
    {
        unsigned char nVertex[3];
        unsigned char nLightNormalIndex;
    };
    struct stMD2Frame
    {
        float fScale[3];
        float fTranslate[3];
        char szName[16];
        stMD2Vertex verts[1];
    };

    unsigned char pBuffer[30000];
    stMD2Frame* pTmp = (stMD2Frame*) pBuffer;

    fseek( pFile, m_Header.nOffsetFrames, SEEK_SET );
    for( int i = 0; i < m_Header.nNumFrames; i++ )
    {
        fread( pTmp, 1, m_Header.nFrameSize, pFile );

        m_pFrames[i].pVerts = new MD2Vertex[m_Header.nNumVertices];
        strcpy_s( m_pFrames[i].szName, pTmp->szName );

        for( int j = 0; j < m_Header.nNumVertices; j++ )
        {
            m_pFrames[i].pVerts[j].fVertex[0] = pTmp->verts[j].nVertex[0] *
                pTmp->fScale[0] + pTmp->fTranslate[0];

            m_pFrames[i].pVerts[j].fVertex[2] = -1 * (pTmp->verts[j].nVertex[1] *
                pTmp->fScale[1] + pTmp->fTranslate[1]);

            m_pFrames[i].pVerts[j].fVertex[1] = pTmp->verts[j].nVertex[2] *
                pTmp->fScale[2] + pTmp->fTranslate[2];
        }
    }

    return true;
}

调试期间转储的变量:

nNumSkins        1          int
nNumVertices     583        int
nNumUV           582        int
nNumTriangles    422        int
nNumFrames       1          int

(我最终删除了我的D3DXVECTOR3结构,所以它有点怪......)

好的。所以,在'for'循环中,它就是爆炸的地方。 如果我这样做:

//  unsigned char pBuffer[30000];
//  stMD2Frame* pTmp = (stMD2Frame*) pBuffer;

    fseek( pFile, m_Header.nOffsetFrames, SEEK_SET );
    for( int i = 0; i < m_Header.nNumFrames; i++ )
    {
        stMD2Frame* pTmp = new stMD2Frame();
        fread( pTmp, 1, m_Header.nFrameSize, pFile );

        m_pFrames[i].pVerts = new MD2Vertex[m_Header.nNumVertices];
        strcpy_s( m_pFrames[0].szName, pTmp->szName );

在分配“m_pFrames [i] .pVerts”语句时,我收到了bad_alloc异常。 有时,我没有得到它,但是当我尝试新建我的D3D课程时,我确实得到了例外(我假设无论后来我做什么,我都会得到它)。 我的MEM使用率非常低,所以它可能是堆损坏吗?

在动画期间创建我的VertexBuffer时,我实际上必须这样做。

我最终应该使用向量吗?我知道他们分配的不仅仅是需要,但它似乎是(最明显的)[最佳]方式。

1 个答案:

答案 0 :(得分:2)

我会检查一下:

strcpy_s( m_pFrames[0].szName, pTmp->szName );

如果从文件加载的字符串不是以空值终止的,那么此副本最终可能会覆盖您的堆栈空间并导致损坏。

修改

再看一下,我认为问题在于您将stMD2Frame定义为

struct stMD2Frame
{
    float fScale[3];
    float fTranslate[3];
    char szName[16];
    stMD2Vertex verts[1];
};

但这只有一个顶点的空间。

当您在框架中阅读

fread( pTmp, 1, m_Header.nFrameSize, pFile );

你会破坏你的记忆。

我建议检查一下

sizeof(stMD2Frame)>=m_Header.nFrameSize 

在阅读数据之前。