由一个内存错误发现的cocos2dx 引擎3.4版本的 一个bug

错误症状

最近做实训cocos2dx项目的TA，一个同学在场景切换时遇到如下问题，场景B->C,C->B,B->C,然后场景C->B，返回场景B时程序就崩溃了

主要涉及代码

bool StartScene::init() {      if (!Scene::init())        return  false;    MenuItemFont *skipMenuItem =           MenuItemFont::create("skip",this, menu_selector(StartScene::SkipMenuItemCallback));    skipMenuItem->setAnchorPoint(Vec2(0, 0));    skipMenuItem->setPosition(Vec2(900, 25));    Menu *skipMenu = Menu::create(skipMenuItem, NULL);    skipMenu->setAnchorPoint(Vec2::ANCHOR_BOTTOM_LEFT);    skipMenu->setPosition(0, 0);    this->addChild(skipMenu);    return true;

回调函数

void StartScene::SkipMenuItemCallback(Ref* sender) {auto levelChosenScene = LevelChooseScene::createScene();Director::getInstance()->pushScene(levelChosenScene);}

bool LevelChooseScene::init() {    if (!Scene::init())        return  false;    CCMenuItemFont *levelChosenMenuItem = CCMenuItemFont::create("LevelChosen", this, menu_selector(LevelChooseScene::LevelChosenMenuItemCallback));    levelChosenMenuItem->setAnchorPoint(Vec2(0, 0));    levelChosenMenuItem->setPosition(Vec2(500, 300));    CCMenu *levelChosenMenu = CCMenu::create(levelChosenMenuItem, NULL);    levelChosenMenu->setAnchorPoint(Vec2::ANCHOR_BOTTOM_LEFT);    levelChosenMenu->setPosition(0, 0);    this->addChild(levelChosenMenu);    return true;}

点击回调函数：

void LevelChooseScene::LevelChosenMenuItemCallback(Ref* sender) {    Director::getInstance()->popScene();}

错误定位

通过简单调试，发现startScene的引用计数异常，在一创建之后就为2，而在第二次pop之后变为0，而另外一个TA发现把startScene中的menuitem用CC_CALLBACK_1创建而不是menu_selector创建的话就没问题

MenuItemFont *skipMenuItem = MenuItemFont::create("skip", CC_CALLBACK_1(StartScene::SkipMenuItemCallback,this));

发现由原来的函数创建时:

bool MenuItemFont::initWithString(const std::string& value, Ref* target, SEL_MenuHandler selector){    CCASSERT( !value.empty(), "Value length must be greater than 0");    _target = target;    CC_SAFE_RETAIN(target);    return initWithString(value, std::bind(selector,target, std::placeholders::_1) );}

会维护了target指针，此例中就是StartScene指针
然后就是定位_target何时release(),

因为scene继承Ref,在Ref的release()函数中增加语句

    if (!strcmp(typeid(*this).name(), "class StartScene"))    {        CCLOG("hehe");    }

然后把断点设在CCLOG处，观察堆栈帧

发现在menuitem::onExit()函数中

会realse()一次，导致了错误的发生
因为每次切换场景都对scene额外多realse()了一次，引擎的bug在于不应该在menuitem中维护target指针

这个问题已经在最新版的cocos2dx引擎中解决了，新版本中不在提供menu_item 维护target指针的create函数重载