cocos2dx 2.x版本在android下CCLabelTTF的一个bug

cocos2dx在android下是采用Paint来生成图片然后在CCLabelTTF里显示的,它具体的代码都在java类Cocos2dxBitmap里,生成完成之后会调用一个jni函数将结果传给cpp层,cpp层靠一个static变量来与java层交换数据,具体如下

       BitmapDC &dc = sharedBitmapDC();

        CC_BREAK_IF(! dc.getBitmapFromJava(pText, nWidth, nHeight, eAlignMask, pFontName, nSize));

        // assign the dc.m_pData to m_pData in order to save time
        m_pData = dc.m_pData;
        CC_BREAK_IF(! m_pData);

这里有个问题,sharedBitmapDC是一直共用的,它的dc.m_pData永远保存的是上一次的数据,如果某种原因java层调用的时候失败了,那么cpp层继续的话,就会拿到上上次的数据,但是这个数据是由cpp层负责free的,于是就会出现double free导致程序崩溃。我们在实际中就碰到了这个情况,当时崩溃的栈告诉我们的是CCImage析构里出现问题

CCImage::~CCImage()
{
    CC_SAFE_DELETE_ARRAY(m_pData);
#if (CC_TARGET_PLATFORM == CC_PLATFORM_WINRT) || (CC_TARGET_PLATFORM == CC_PLATFORM_WP8)
    CC_SAFE_DELETE(m_ft);
#endif
}

可是单纯看CCImage的代码是看不问题的,而且如果这里有问题,那麻烦大了。最终检查了之后还是确认cpp部分可能会隐藏bug,于是将上面CCImage里修改如下

        BitmapDC &dc = sharedBitmapDC();

        CC_BREAK_IF(! dc.getBitmapFromJava(pText, nWidth, nHeight, eAlignMask, pFontName, nSize));

        // assign the dc.m_pData to m_pData in order to save time
        m_pData = dc.m_pData;
        dc.m_pData = NULL;
        CC_BREAK_IF(! m_pData);

这样一来,cpp层肯定不会再出现double free了,于是再测,这个时候发现java层抛异常了,之前的异常由于程序退出根本就没打印得出来。再去追查java层,发现具体原因是

setContentSize之后,java层根据ContentSize判断出来,字体根本放不进去,于是它认为创建出来的Bitmap的高度是0,而Bitmap根本不允许创建高度为0的,于是异常了。我解决的办法是至少让它显示出来一行一列,这样我们看到显示就知道哪里出问题了,而不是直接崩溃。

从这个问题再一次印证了,fail early, fail loud的编程习惯,否则一个简单问题就变成复杂问题了。

郑重声明:本站内容如果来自互联网及其他传播媒体,其版权均属原媒体及文章作者所有。转载目的在于传递更多信息及用于网络分享,并不代表本站赞同其观点和对其真实性负责,也不构成任何其他建议。