Segmentation fault

**(Slith++)** · 05-06-2007

I cant figure out why this seg faults's.

Main.cpp

Code:

Charactor *ch = new Charactor();


int main(int argc, char** argv) {
ch->addAnimation("WalkRight",sp);
}
}

Charactor.h

Code:

#include <map>
class Charactor{
    public:
        Charactor();
        void addAnimation(char *animationAlias, Sprite *sp);
        std::map<const char*, Sprite*, strCmp> *animations;
};

Charactor.cpp

Code:

#include "Charactor.h"

Charactor::Charactor(){
    
}
void Charactor::addAnimation(char *animationAlias, Sprite *sp){ 
    (*this->animations)[animationAlias] = sp;
}

And its not because sp is NULL.

Thanks

**vart** · 05-06-2007

probably - because animations is not initialized pointer...

Why do you need pointers at all?

**grumpy** · 05-06-2007

It is usually a good idea to give a small but complete example that illustrates your problem. What you've obviously done is remove things you think are irrelevant, so the code you've supplied should not even compile, let alone yield a segmentation fault when run.

All that is possible is therefore a wild guess. My wild guess is that your Charactor constructor does not initialise the animations member to point at anything, so dereferencing it yields undefined behaviour.

**(Slith++)** · 05-06-2007

Originally Posted by grumpy

My wild guess is that your Charactor constructor does not initialise the animations member to point at anything, so dereferencing it yields undefined behaviour.

Im pretty sure this is what is wrong also. What do I have to put in my Charactor constructor?

**vart** · 05-06-2007

Why not to use objects instead of pointers?

Thread: Segmentation fault

Thread Tools

Search Thread

Display