Question

昨天我花了一些时间调试一个可爱的Heisenbug，xmlNodePtr的类型会改变我。此示例显示错误：

#include <iostream>

#include <vector>
#include <string>
#include <memory>    // std::unique_ptr

#include <cstdint>

#include <libxml/tree.h>
#include <libxml/parser.h>

struct SomeDataType {
    std::vector<std::vector<std::string>> data;

    explicit SomeDataType(uint32_t rows_, uint32_t columns_)
        : data(rows_)
    {
        for (uint32_t row = 0; row < rows_; ++row) {
            data[row].resize(columns_);
        }
    }
};

static std::vector<xmlNodePtr> GetChildren(xmlNodePtr node)
{
    std::vector<xmlNodePtr> children;

    xmlNodePtr child = node->children;
    while (child) {
        if (child->type == XML_ELEMENT_NODE) {
            children.push_back(child);
        }
        child = child->next;
    }

    return children;
}

int main() {
    std::unique_ptr<xmlDoc, void(*)(xmlDoc*)> document = { xmlParseEntity("libxml2-fail.xml"), xmlFreeDoc };

    SomeDataType{ 3, 2 };

    xmlNodePtr root = xmlDocGetRootElement(document.get());

    for (const xmlNodePtr &child : GetChildren(root)) {
        const xmlNodePtr &entry = GetChildren(child)[0]; // Problem here...
        std::cout << "Expected " << XML_ELEMENT_NODE << " but was " << entry->type << std::endl;
        std::cout << entry->name << std::endl;
    }
}

编译：

g++ -g -std=c++14 -Wall -Wextra -pedantic -I/usr/include/libxml2 libxml2-fail.cpp -lxml2 -o fail.out

xml文件：

<?xml version="1.0" encoding="utf-8"?>
<data>
  <tag>
    <subtag>1</subtag>
  </tag>
</data>

Running给出了以下输出：

Expected 1 but was 17

逐步使用gdb，一切正常，直到我们到达const xmlNodePtr & = ...行。它没有类型XML_ELEMENT_NODE，而是类型XML_ENTITY_DECL。但是，如果我运行以下命令，引用xmlNodePtr将变为我期望的类型：

48          const xmlNodePtr &entry = GetChildren(child)[0];
(gdb) n
49          std::cout << "Expected " << XML_ELEMENT_NODE << " but was " << entry->type << std::endl;
(gdb) p *entry
$1 = {_private = 0x0, type = XML_ENTITY_DECL, name = 0x0, children = 0xb7e67d7c <std::string::_Rep::_S_empty_rep_storage+12>, last = 0x0, parent = 0x69, next = 0x0, prev = 0x9, doc = 0x0, ns = 0x805edb8, content = 0x805edb8 "", properties = 0x0, nsDef = 0x0, psvi = 0x0, line = 60648, extra = 2053}
(gdb) p *child
$2 = {_private = 0x0, type = XML_ELEMENT_NODE, name = 0x805ee98 "tag", children = 0x805eea8, last = 0x805ef98, parent = 0x805edb8, next = 0x805efe8, prev = 0x805ee08, doc = 0x805ece8, ns = 0x0, content = 0x0, properties = 0x0, nsDef = 0x0, psvi = 0x0, line = 3, extra = 0}
(gdb) p GetChildren(child)
$3 = std::vector of length 1, capacity 1 = {0x805eef8}
(gdb) p *entry
$4 = {_private = 0x0, type = XML_ELEMENT_NODE, name = 0x805ef38 "subtag", children = 0x805ef48, last = 0x805ef48, parent = 0x805ee58, next = 0x805ef98, prev = 0x805eea8, doc = 0x805ece8, ns = 0x0, content = 0x0, properties = 0x0, nsDef = 0x0, psvi = 0x0, line = 4, extra = 0}
(gdb)

当我像这样循环遍历一个元素时，我没有问题：

for (const xmlNodePtr &entry : GetChildren(child)) {
    ...
}

当我不像xmlNodePtr那样使用const引用时，我也没有问题：

xmlNodePtr entry = GetChildren(child)[0];

然而，根据this stackoverflow question，它应该不是问题。

SomeDataType结构是奇怪的必要;否则我得到一个段错误，因为entry变成空指针。

这个错误是什么来的？

Answer 1

执行此操作时：

const xmlNodePtr &entry = GetChildren(child)[0]; // Problem here...

您实际上是以一种非终身扩展的方式绑定对临时的引用。 operator[]返回一个引用，因此您不绑定对临时引用的引用 - 您绑定对引用的引用。但是从operator[]返回的引用引用了vector返回的基础临时 GetChildren()中的一个元素，该元素在行尾超出范围，离开你自己一个晃来晃去的参考。

然而，当您尝试：

时

for (const xmlNodePtr &entry : GetChildren(child)) {

这是语法糖：

{
    auto&& __range = GetChildren(child); // bind temporary to reference
                                         // lifetime IS extended
    auto b = begin(__range);
    auto e = end(__range);
    for (; b != e; ++b) {
        const xmlNodePtr& entry = *b;
        // ...
    }
}

这里，*b不是临时的或临时的任何部分 - 它是对容器的引用，其生命周期与__range一样长，这是通过循环的整个主体。没有悬空参考。

类似地，

xmlNodePtr entry = GetChildren(child)[0];

只是复制，没有任何参考问题。

当我观察它时，libxml2 xmlNodePtr的类型会发生变化

1 个答案: