SIGSEGV sur une std::map::find();

Bonjour !

Ca fait un petit peu de temps que je suis sur cette erreur, et je vois pas du tout vers ou me tourner pour resoudre ca.

Voici la fonction qui m'envoie ce demon ! (En gras, l'erreur SIGSEGV)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

void Clips::addClip(string g, SDL_Rect* r) { if(_spr->rects.find(g) == _spr->rects.end()) { _spr->rects.insert(make_pair(g, vector<SDL_Rect*>())); } _spr->rects.find(g)->second.push_back(r); ++_spr->scount; if(_spr->index < 0) { _spr->index = 0; } _spr->ci = _spr->rects[g].begin(); }

---

Voici egalement toute la definition, ainsi que le constructeur

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27

struct __sprite { map<string, vector<SDL_Rect*>> rects; string group; int index; int scount; int gcount; bool loop; // true = animate to RIGHT, false = to LEFT bool _autoanimate; vector<SDL_Rect*>::iterator ci; }; class Clips { public: Clips(); void addClip(string g, SDL_Rect* r); void loopTo(bool dir) { _spr->loop = dir; } void autoanimation(bool anim) { _spr->_autoanimate = anim; } bool autoanimate() { return _spr->_autoanimate; } void refreshiterator(); SDL_Rect* get(); void setgroup(string g) { _spr->group = g; refreshiterator(); } string group() { return _spr->group; } bool hoverable() { return !_spr->_autoanimate; } private: void nextclip(); __sprite* _spr; };

---

Constructeur:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

Clips::Clips() {   shared_ptr<__sprite> _spr(new __sprite); _spr->_autoanimate = false; _spr->loop = true; _spr->scount = 0; _spr->gcount = 0; _spr->index = -1; _spr->group = ""; }

---

Voici également le resultat du débug !

Citation:

---

#0 0046D083 std::_Rb_tree<std::string, std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > >, std::_Select1st<std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > > >, std::less<std::string>, std::allocator<std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > > > >::_M_begin(this=0xbaadf00d) (c:/program files (x86)/codeblocks/mingw/bin/../lib/gcc/mingw32/4.4.1/include/c++/bits/stl_tree.h:482)
#1 0046CF64 std::_Rb_tree<std::string, std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > >, std::_Select1st<std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > > >, std::less<std::string>, std::allocator<std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > > > >::find(this=0xbaadf00d, __k=@0x28fbb4) (c:/program files (x86)/codeblocks/mingw/bin/../lib/gcc/mingw32/4.4.1/include/c++/bits/stl_tree.h:1421)
#2 00469995 std::map<std::string, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> >, std::less<std::string>, std::allocator<std::pair<std::string const, std::vector<SDL_Rect*, std::allocator<SDL_Rect*> > > > >::find(this=0xbaadf00d, __x=@0x28fbb4) (c:/program files (x86)/codeblocks/mingw/bin/../lib/gcc/mingw32/4.4.1/include/c++/bits/stl_map.h:659)
#3 00404132 Clips::addClip(this=0x915238, g={static npos = 4294967295, _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>}, _M_p = 0x915284 "stand"}}, r=0x915258) (D:\C++\projects\Client v2\Factories\Tools.cpp:16)
#4 00406072 __WFSprite(this=0x9150b8, spritesheet={static npos = 4294967295, _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>}, _M_p = 0x28fca8 "\224P\221"}}, p={x = 200, y = 200}, s={x = 80, y = 140}) (D:\C++\projects\Client v2\Factories\Widgets\WFSprite.cpp:16)
#5 00423F98 WidgetFactory::Create<__WFSprite>(this=0x914d30, spritesheet={static npos = 4294967295, _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>}, _M_p = 0x915094 "sprite.bmp"}}, p={x = 200, y = 200}, s={x = 80, y = 140}) (D:/C++/projects/Client v2/Factories/WidgetFactory.h:29)
#6 00408516 GUIManager::Show(this=0x913480) (D:\C++\projects\Client v2\GUIManager.cpp:61)
#7 00408903 SDL_main(argc=1, argv=0x912fa0) (D:\C++\projects\Client v2\main.cpp:14)
#8 00409E49 console_main(argc=1, argv=0x912fa0) (./src/main/win32/SDL_win32_main.c:210)
#9 00000000 0x00000001 in ??() (??:??)
#10 00000000 0x0000001b in ??() (??:??)
#11 00000000 0x00000000 in ??() (??:??)

---

Apparement, le programme essaye d'acceder a la map, alors qu'elle n'a pas de memoire ?!
Pourtant, jj'ai lu un truc sur google (d'où le shared_ptr !)

Dans la classe, _spr est un pointeur normal. Dans le constructeur tu créé un shared_ptr, qui est détruit à la fin du constructeur, et la sprite avec puisqu'il n'y a pas d'autre shared_ptr dessus.

Bonjour,

Tu n'initialise pas la variable membre _spr dans ton constructeur (d'aiileurs pourqui utiliser un pointeur ? pour faire un pimpl ?), c'est une variable locale du même nom qui est initialisée.
_spr pointe donc vers n'importe où et dès que tu essayes d'y accéder tu as un segmentation fault. Pour ce genre d'erreur, "0xbadf00d" peut te mettre la puce à l'oreille ( voir ici )

J'ai enlevé le pointeur, est ca supprime le SIGSEGV (apparement)


Faux espoir.


Le SIG* se déplace, et je remonte plus haut dans le programme !
Il s'est transformé en SIGILL

Le premier appel se trouve ici:

Code:

---

WidgetFactory::getInstance()->Create<__WFSprite>((string)"sprite.bmp", oPos({200,200}), oSize({80,140}))

---

Voici le template utilisé

Code:

---

1 2 3 4 5

template<typename _WidgetClass> pair<string, Widget*> Create(string spritesheet, oPos p, oSize s) { return make_pair(UniqueID(), new _WidgetClass(spritesheet, p, s)); }

---

MArche bien également.
donc, la je fais en locurence un new __WFSprite(string, opos, osize);

Voici le constructeur de __WFSPrite

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

__WFSprite::__WFSprite(string spritesheet, oPos p, oSize s) { // surface SDL_Surface* surf = load_image(spritesheet); // size, position SDL_Rect* sandp = CreateRect(s.x, s.y, p.x, p.y); // sprite sheet cut SpriteSheet clips; // for now // groups = 1 // clips = 4 for(int i = 0; i < 4 ; ++i) { clips.addClip("stand", CreateRect(i*sandp->w+i*1, 0, sandp->w, sandp->h)); } clips.autoanimation(true); // creating widget Widget(UniqueID(), surf, NULL, sandp, &clips, NULL, false, true); // SIGILL char t[50]; sprintf(t, "size of clips is %d", _ss->getgroup()); MessageBox(NULL, t, "sprite", MB_OK); }

---

Note: __WFSprite herite publiquement de Widget, donc voici le header de Widget:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48

class Widget { public: //ctor Widget(string id, SDL_Surface* sprite, SDL_Surface* dest, SDL_Rect* sandp, SpriteSheet* c, funcptr callback, bool focusable, bool autoanimate) : _id(id), _sprite(sprite), _dest(dest), _sandp(sandp), _ss(c), _callback(callback), _focusable(focusable) {}; // var set void SetDest(SDL_Surface* dst); void SetCallback(funcptr fptr); // clip call string ClipState(); void MouseOver(); void MouseOut(); void MouseClick(); // focus |callback bool CallBack() { return _callback(); } virtual bool hasFocus() {} virtual void GetFocus() {} virtual void DelFocus() {} virtual void Animate() {} bool Focusable() { return _focusable; } bool isSprite() { return (!_focusable && (_callback == NULL)); } // gfx void Draw(); // return widgets rect SDL_Rect* rect() { return _sandp; } string id() { return _id; }   virtual ~Widget();   protected: Widget() {}; string _id; SDL_Surface* _sprite; SDL_Surface* _dest; SDL_Rect* _sandp; SpriteSheet* _ss; funcptr _callback; bool _focusable; private: };

---

donc je me retrouve avec un constructeur, apparament illégal ? :(

Si tu as plusieurs fichiers source dans ton projet, effaces les fichiers intermédiaires de compilation et recompiles a partir de rien (make clean && make)