[SOLVED]sf::Sprite::SetImage() crash

[Masadow]

Hi,

I searched for hours but i can't find why this code crash

Code: [Select]

ge::Menu::Menu(std::string bg_path)
{
m_bg = new sf::Sprite;

std::cout << "receive addr " << ge::ResourceManager::getInstance()->getImage(bg_path) << std::endl;
sf::Image &test = *ge::ResourceManager::getInstance()->getImage(bg_path);
m_bg->SetImage(test);
}

Code: [Select]

sf::Image* ge::ResourceManager::getImage(std::string path)
{
Item *it;

if ((it = search(IMAGE, path)))
{
return (sf::Image*)(it->ptr);
}
sf::Image img;
if (img.LoadFromFile(path))
{
sf::Image *i = (sf::Image*)insert(&img, IMAGE, path)->ptr;
std::cout << "Create image at addr " << i << std::endl;
return i;
}
return NULL;
}


And the console output

Create image at addr 0x28fb40
receive addr 0x28fb40
cast addr 0x28fb40

The crash seems to be a Segmentation Fault on sf::Sprite::SetImage() call.
Someone can give me a tip ? Or more code is needed ?

Thanks in advance

[slotdev]

Is this SFML 2?

You're using "new" to instantiate a new Sprite. Yuk!! Not good. I would sort that out first of all.

[Masadow]

Yeah, sf::Sprite pointer was a test, forgot to change :p

I use SFML 1.6

Maybe i should give you the Item struct returned by insert().

Code: [Select]

struct Item
{
void *ptr;
std::string name;
enum Type type;
struct Item *next;
};

[Laurent]

Code: [Select]

sf::Image *i = (sf::Image*)insert(&img, IMAGE, path)->ptr;

You store the address of a variable that will no longer exist after the function returns. So when you try to use it later, it crashes.

[Masadow]

uh... how can i be so stupid :p


Merci Laurent x)