通过三指针在另一个函数中分配内存,sscanf异常

时间:2011-12-21 21:22:45

标签: c++ pointers memory-management nullpointerexception scanf

我将内存分配给另一个函数中的双指针,因此我需要使用指向指针的指针。当我使用sscanf时,我得到一个例外,我不确定为什么。下面是一段代码。当它全部在同一个函数中并且我只需要使用双指针时,它工作得更早,但现在我重构代码并使用三重指针即可解决这个问题。

typedef float vector[3]

int mainLoaderFunc() {

    char* memory = NULL;
    size_t size = loadFile(fileName, &memory); // load model file into memory, this works, tested and true

    // create vector arrays
    vector *vertexArray = NULL;         
    vector *normalArray = NULL;         
    vector *textureArray = NULL;        

    loadArrays(size, memory, &vertexArray, &normalArray, &textureArray);

    // do other stuff with arrays

}

void loadArrays(size_t size, char *memory, vector **vertexArray, vector **normalArray, vector **textureArray) {

    int numVerts = 0; 
    int numNormals = 0;  
    int numTextures = 0;  

    char* p = memory;           // pointer to start of memory
    char* e = memory + size;    // pointer to end of memory

    // count verts, normals, textures for memory allocation
    while (p != e) {
        if (memcmp(p, "vn", 2) == 0) {
            numNormals++;
        } else if (memcmp(p, "vt", 2) == 0) { 
            numTextures++;
        } else if (memcmp(p, "v",  1) == 0) {
            numVerts++;
        } 
        while (*p++ != (char) 0x0A);
    }

    // allocate memory for vector arrays
    *vertexArray        = new vector[numVerts];
    *normalArray        = new vector[numNormals];
    *textureArray       = new vector[numTextures];

    p = memory;

    int vertexIndex = 0;
    int normalIndex = 0;
    int textureIndex = 0;  //*** IF BREAK POINT HERE: NO EXCEPTION

    // load data from memory into arrays
    while (p != e) {

        if (memcmp(p, "vn", 2) == 0) {
            sscanf(p, "vn %f %f %f", &normalArray[normalIndex][0], &normalArray[normalIndex][1], &normalArray[normalIndex][2]);
            normalIndex++;
        } else if (memcmp(p, "vt", 2) == 0) {
            sscanf(p, "vt %f %f", &textureArray[textureIndex][0], &textureArray[textureIndex][1]);
            textureIndex++;
        } else if (memcmp(p, "v", 1) == 0) {
            sscanf(p, "v %f %f %f", &vertexArray[vertexIndex][0], &vertexArray[vertexIndex][1], &vertexArray[vertexIndex][2]);
            vertexIndex++;
        } 
        while (*p++ != (char) 0x0A);
    }

}

一旦代码命中sscanf部分,我就会得到例外:

Unhandled exception at 0x5e9cf2dc (msvcr100d.dll) in derp.exe: 0xC0000005: Access violation writing location 0xcccccccc.

3 个答案:

答案 0 :(得分:2)

您将无效地址传递给sscanf。看看你的一个函数的参数:

vector **vertexArray

vertexArray指向从mainLoaderFunc传递的指针的地址。 *vertexArray指向实际vector数组的指针,正如您为它分配内存时所意识到的那样:

*vertexArray = new vector[numVerts];

因此,数组中的有效元素为:

(*vertexArray)[vertexIndex][0]

请注意您的代码遗失的*; *vertexArray指的是实际的数组。现在sscanf想要一个这个变量的地址,所以现在得到它的地址(为了清楚起见,添加了额外的括号):

&( (*vertexArray)[vertexIndex][0] )

将所有参数更改为sscanf,就像这样。

答案 1 :(得分:1)

有人指出,你真的不应该在这里使用三重指针。

此外,您可能希望将文件扫描作为链接if-blocks之外的其他内容(或者至少将其拆分为函数)。

这可以解决您的问题,因为您只有2个解除引用,您应该有3:

    if (memcmp(p, "vn", 2) == 0) {
        sscanf(p, "vn %f %f %f", &(*normalArray)[normalIndex][0], &(*normalArray)[normalIndex][1], &(*normalArray)[normalIndex][2]);
        normalIndex++;
    } else if (memcmp(p, "vt", 2) == 0) {
        sscanf(p, "vt %f %f", &(*textureArray)[textureIndex][0], &(*textureArray)[textureIndex][1]);
        textureIndex++;
    } else if (memcmp(p, "v", 1) == 0) {
        sscanf(p, "v %f %f %f", &(*vertexArray)[vertexIndex][0], &(*vertexArray)[vertexIndex][1], &(*vertexArray)[vertexIndex][2]);
        vertexIndex++;
    } 

答案 2 :(得分:1)

将传递指针传递给sscanf()时,您没有正确取消引用它们。试试这个:

if (memcmp(p, "vn", 2) == 0) { 
    sscanf(p, "vn %f %f %f", &((*normalArray)[normalIndex][0]), &((*normalArray)[normalIndex][1]), &((*normalArray)[normalIndex][2])); 
    ++normalIndex; 
} else if (memcmp(p, "vt", 2) == 0) { 
    sscanf(p, "vt %f %f", &((*textureArray)[textureIndex][0]), &((*textureArray)[textureIndex][1])); 
    ++textureIndex; 
} else if (memcmp(p, "v", 1) == 0) { 
    sscanf(p, "v %f %f %f", &((*vertexArray)[vertexIndex][0]), &((*vertexArray)[vertexIndex][1]), &((*vertexArray)[vertexIndex][2])); 
    ++vertexIndex; 
}