Question

所以我最近得到了一个RapidXML用作在我的程序中解析XML的方法，我主要使用它作为一种混乱的方式，但我已经得到了一些非常奇怪的问题，我在这里真的很难追查。试着通过这个来坚持我，因为我试图解决这个问题非常彻底，但我必须遗漏一些东西。

首先是这里的XML：

<?xml version="1.0" encoding="utf-8" ?>
<resources>
    <image key="tilemap_roguelikesheet" path="res/media/tilemaps/roguelikesheet.png" />
    <image key="tilemap_tiles" path="res/media/tilemaps/tiles.png" />
</resources>

发生段错误的功能：

void TextureManager::LoadResource(const char* pathToFile)
{
    rapidxml::xml_document<>* resource = Resources::LoadResource(pathToFile);
    std::string imgName;
    std::string imgPath;

    if (resource != NULL)
    {
        rapidxml::xml_node<>* resourcesNode = resource->first_node("resources");

        if (resourcesNode != NULL)
        {
            for (rapidxml::xml_node<>* child = resourcesNode->first_node("image"); child; child = child->next_sibling())
            {
                //Crash here on the second loop through.
                imgName = child->first_attribute("key")->value();
                imgPath = child->first_attribute("path")->value();
                Astraeus::Log(moduleName, "Image Name: " + imgName);
                Astraeus::Log(moduleName, "Image Path: " + imgPath);

                TextureManager::AddTexture(imgName, imgPath);
            }
        }
        else
        {
            Astraeus::Error(moduleName, "Resources node failed to load!");
        }

        resource->clear();

    }
    else
    {
        std::string fileName(pathToFile);
        Astraeus::Error(moduleName, fileName + " could not be loaded.");
    }
}

因此，在for循环的第二个循环中发生段错误以遍历所有节点，并在尝试执行imgName赋值时触发。事情变得有些奇怪。在对程序进行调试时，初始子节点细分显示它具有指向下一个节点及其元素/属性等的内存指针。在调查这些节点时，您可以看到值存在且rapidxml看似成功解析了文件。

然而，当第二个循环发生时，显示child仍然具有完全相同的内存指针，但这次值的细分显示它们本质上是NULL值，因此程序失败并且我们得到代码139。如果你试着看看前面的节点，我们刚来的那些值也是NULL。

现在说，我注释掉了调用AddTexture函数的行，该节点能够打印出所有节点值，完全没有问题。（Log方法基本上只是打印到控制台，直到我用它做一些更时髦的东西。）所以问题必须在于函数？这是：

void TextureManager::AddTexture(const std::string name, const std::string path)
{

    Astraeus::Log(moduleName, "Loading texture: " + path);
    if (texturesLookup.find(name) != texturesLookup.end())
    {
        Astraeus::Error(moduleName, "Texture Key: " + name + " already exists in map!");
    }
    else
    {

        texturesLookup.insert(std::make_pair(name, path));
        //Texture* texture = new Texture();

        /*if (texture->LoadFromFile(path))
        {
           //textures.insert(std::make_pair(name, texture));
        }
        else
        {
            Astraeus::Error(moduleName, "Failed to add texture " + name + " to TextureManager!");
        }*/
    }
}

忽略字符串传递的事实，因此不应该以任何方式影响节点，这个函数仍然有点不确定。如果我注释掉它可以工作的一切，但有时候会再次崩溃。一些代码被注释掉了，因为我没有直接添加键名，加上一个指向纹理的内存指针，而是切换到存储键和路径字符串，然后我可以稍后将纹理加载到内存中作为变通方法。这个解决方案起了一点作用，但肯定会再次开始逐段崩溃。

我无法真正可靠地复制或缩小每次导致问题的原因，所以非常感谢任何帮助。 RapidXML doc是否会超出范围或被删除？

对于记录，类实际上只是静态的，以及存储纹理指针的地图。

谢谢！

Answer 1

我不确定，但我认为Martin Honnen说明了这一点。

如果next_sibling()将指针返回到两个“image”元素之间的文本节点，则在写入时

imgName = child->first_attribute("key")->value();

您获得child->first_attribute("key")是空指针，因此->value()取消引用空指针。崩溃！

我想你应该得到next_sibling("image")元素;

之类的东西

for (rapidxml::xml_node<>* child = resourcesNode->first_node("image");
     child;
     child = child->next_sibling("image"))

为了确保不使用空指针，我强烈建议你检查属性指针（你是否非常确定那个“图像”元素带有“密钥”和“路径” “元素？）;像这样的东西

         if ( child->first_attribute("key") )
            imgName = child->first_attribute("key")->value();
         else
            ;  // do something

         if ( child->first_attribute("path") )
            imgPath = child->first_attribute("path")->value();
         else
            ;  // do something

p.s：抱歉我的英语不好。

Answer 2

这条线正在咬牙切齿......

 rapidxml::xml_document<>* resource = Resources::LoadResource(pathToFile);

LoadResource会返回一个指针，但你永远不会把它释放到任何地方......？

你是否100％确定该函数不会返回指向现在超出范围的对象的指针。像这个经典的错误...

int * buggy()
{
  int i= 42;
  return &i;   // UB
}

正如@ max66所说。您应该使用next_sibling("image")。如果失败了，你需要找出原因。

Answer 3

所以对于将来再次回来的人来说，这就是发生的事情。

是的，这是一个范围问题，但不是因为我一直在思考的xml_document。资源加载函数中的xml_file变量超出了范围，这意味着由于RapidXML将内容存储在内存中的方式，一旦超出范围，它就会释放内存，从而导致下一次动态分配是由一个特定的函数发生的，它会搞砸xml文档并用垃圾数据填充它。

所以我想最好的想法是确保xml_file和xml_document不会超出范围。我已经添加了以前答案中的一些建议，但是在删除以帮助调试过程之前，我将在代码中指出这些项目。

感谢大家的帮助/建议。

RapidXML访问兄弟节点导致段错误似乎没有理由

3 个答案: