Passing pointer crash my program

[DirtyBlasion]

So I'm trying to make elements from an array know about their neighbors, so they can interfer with each other.
Here I initialize them:

    for(int x = 0; x < SCX; x++)
        for(int y = 0; y < SCY; y++)
            for(int z = 0; z < SCZ; z++)
            {
                chunks[x][y][z] = new Chunk(texPath, x, y, z);
                //top
                if(CheckForBounds(x, y+1, z))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_FRONT, chunks[x][y+1][z]);
                }
                //bottom
                if(CheckForBounds(x, y-1, z))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_BOTTOM, chunks[x][y-1][z]);
                }
                //left
                if(CheckForBounds(x-1, y, z))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_LEFT, chunks[x-1][y][z]);
                }
                //right
                if(CheckForBounds(x+1, y, z))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_RIGHT, chunks[x+1][y][z]);
                }
                //front
                if(CheckForBounds(x, y, z+1))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_FRONT, chunks[x][y][z+1]);
                }
                //back
                if(CheckForBounds(x, y, z-1))
                {
                    chunks[x][y][z]->SetNeighbor(chunks[x][y][z]->SIDE_BACK, chunks[x][y][z-1]);
                }
            }
    //chunks[0][0][0]->Test();
}

And the function is:

void Chunk::SetNeighbor(int index, Chunk* neighbor)
{
    Neighbors[index] = neighbor;
    changed = true;
}

the above code just crash my program.

[ne555]

the above code doesn't compile, you need to provide enough code to reproduce your issue.

[DirtyBlasion]

they are delcared like this

//inside the megachunk class
Chunk *chunks[SCX][SCY][SCZ];
//inside the chunk class
Chunk *Neighbors[SIDE_COUNT];

[Moschops]

void Chunk::SetNeighbor(int index, Chunk* neighbor)
{
Neighbors[index] = neighbor;
changed = true;
}

If this code crashes, it is most likely because the array Neighbours is of a size such that there is no element number index; you're trying to write beyond the array. What's the value of index, how big is Neighbors ?

[DirtyBlasion]

enum
{
SIDE_TOP,
SIDE_BOTTOM,
SIDE_LEFT,
SIDE_RIGHT,
SIDE_FRONT,
SIDE_BACK,
SIDE_COUNT
};

in first code you see I'm calling

Chunk *Neighbors[SIDE_COUNT];

but it is strange that it crash even if I change the function in this:

void Chunk::SetNeighbor(int index, Chunk* neighbor)
{
//Neighbors[index] = neighbor;
changed = true;
}

I think the way it is passed crash the program??

[lastchance]

Highly speculative, but your line 9 isn't consistent. Should SIDE_FRONT be SIDE_TOP? May be a possibility of writing outside array bounds if wrong.

Otherwise, it's difficult to tell from the amount of code posted. Will your debugger tell which line it crashed on?

[Moschops]

Also, depending on what CheckForBounds does, you're almost certainly passing bad pointers to the function SetNeighbour.

The very first time, when the only valid pointer in your array is chunks[0][0][0], will SetNeighbour be called?

[DirtyBlasion]

No it's code::blocks, I dont really enjoy VS

Buy it's not a problem from bounds, like I said, if I comment it, it crashes

[Moschops]

I expect you've written over the stack. Trashed some memory.

[DirtyBlasion]

And how I can solve this?

[Thomas1965]

http://www.cprogramming.com/debugging/debugging_strategy.html

What OS / compiler / IDE do you use ?

[DirtyBlasion]

Well I found the mistake, CheckForBounds, was glitched out because of a mistaken paste...