【问题标题】:Copying (using assignment) a structure to a structure inside a union causing seg fault将结构复制(使用赋值)到联合内的结构导致段错误
【发布时间】:2014-05-10 21:18:02
【问题描述】:

我写了以下代码:

#include <iostream>
#include <string>
#include <cstring>

struct bar
{
  std::string s3;
  std::string s4;
}Bar;

union foo
{
  char * s1;
  char * s2;
  bar    b1;

  foo(){};
  ~foo(){};
}Foo;


int main ()
{
  foo f1;
  bar b2;

  std::string temp("s3");
  b2.s3 = temp;
  b2.s4 = temp;

  //f1.b1 = b2;                           //-- This Fails (Seg faults)

  /*
    #0  0x00002b9fede74d25 in std::string::_Rep::_M_dispose(std::allocator<char> const&) [clone .part.12] ()
        from /usr/local/lib64/libstdc++.so.6
    #1  0x00002b9fede75f09 in std::string::assign(std::string const&) () from /usr/local/lib64/libstdc++.so.6
    #2  0x0000000000400ed1 in bar::operator= (this=0x7fff3f20ece0) at un.cpp:5
    #3  0x0000000000400cdb in main () at un.cpp:31
  */

  memcpy( &f1.b1, &b2, sizeof(b2) );  //-- This Works 

  std::cout << f1.b1.s3 << " " << f1.b1.s4 << std::endl;
  return 0;
} 

你能解释一下为什么会出现分段错误吗?我无法解读回溯中的数据所暗示的内容。

【问题讨论】:

  • @Wilding:代码无法编译。
  • 是的。请再检查一次。我是这样编译的:g++ -Wall -g -std=c++11 un.cpp -o u
  • 我在VS2013上编译过。
  • 你能粘贴错误吗?
  • 错误 2 错误 C2039: 'b1' : 不是 'foo' main.cpp 的成员 41 1 错误 3 错误 C2039: 'b1' : 不是 'foo' main.cpp 的成员43 1 错误 4 错误 C2228:'.s3' 左侧必须有类/结构/联合 main.cpp 43 1 错误 5 错误 C2228:'.s4' 左侧必须有类/结构/联合 main.cpp 43 1跨度>

标签: c++ struct memcpy unions


【解决方案1】:

union foo 无法初始化 bar 对象(它如何知道要调用哪个成员的初始化程序?),因此无法初始化 std::strings。如果你想使用 foo 里面的 bar,那么你需要手动初始化它,像这样......

new (&f1.b1) bar; // Placement new
f1.b1 = b2;
// And later in code you'll have to manually destruct the bar, because
//   foo doesn't know to destruct the bar either...
f1.b1.~bar();

或者,您可以尝试自己将此功能滚动到联合的构造函数和析构函数中。

foo() : b1() {}
// Or you construct like this, which you might need to for a non-trivial union...
// foo() { new (&b1) bar; }  // Placement new.
~foo() { b1.~bar(); }

请注意,复制也需要特殊处理。

【讨论】:

  • 你解决了。尽管其他答案对我有很大帮助,但这是问题的实际答案。只需在f1.b1 = b2; 之前添加new (&amp;f1.b1) bar; 即可修复我的代码
  • 请注意,这只是添加放置新泄漏。除非您手动这样做,否则 bar 对象(以及它的字符串)永远不会破坏。
  • 是的,我处理好了。为构造函数和析构函数添加了分配和解除分配..
  • 其实...做析构函数可能不好。如果您将联合用作字符指针,它将使字符串数据无效,这可能对现在初始化和需要销毁的字符串数据造成严重破坏。老实说,如果没有更多关于你想要完成的工作的详细信息,就不可能说出如何处理你的工会。一般来说,联合是非常棘手的对象,具有非常具体的情境需求。
【解决方案2】:

您不能使用 memcpy 复制对象或包含对象的结构,因为它们不会被正确初始化。字符串具有指向 char 数组的指针,如果两个字符串可以共享同一个指针,则必须进行某种垃圾收集(通常是引用计数器)。如果你做 f1.b1 = b2,编译器会生成代码来正确初始化字符串。

【讨论】:

  • 但是莫里斯,我不能这样做f1.b1 = b2,因为 std::string 有一个不平凡的构造函数。我在这里有什么选择?
  • 我认为你应该重新考虑你的数据结构。可以肯定的是,您只能初始化联合的第一个字段。 b1 是 Foo 的第一个字段的提供者,您可以按如下方式初始化 f1: Foo f1 = {Bar()};
  • Wildling,你可以使用 C++11,然后非常、非常、非常、非常仔细地阅读联合的扩展。但是,如果您在 stackoverflow 上询问,您可能不应该使用它。
  • @gasher729 也许吧,但对于工会,我总觉得我没有充分利用它们。
【解决方案3】:

std::string 有一个重要的构造函数,用于初始化其内部成员。因此,您的struct bar 不是 POD 结构。

联合仅支持 POD(这在 C++11 中有所放松)。编译器无法决定调用哪个联合成员的哪个构造函数。想象以下情况:

unition MyUnion {
  std::string s;
  std::vector v;
};

应该使用vector的构造函数还是string的构造函数来初始化对象?

所以在您的情况下,当您将字符串分配给联合的字符串时,内部数据没有初始化,这会导致随机错误。

【讨论】:

  • 另外,我正在使用C++11
  • 这是不可能的。即使在 C++11 中,它也只适用于简单的构造函数。看看 Bjarne Stroustrups C++11 关于这个问题的常见问题解答:stroustrup.com/C++11FAQ.html#unions
猜你喜欢
  • 2014-12-27
  • 1970-01-01
  • 2016-01-21
  • 2018-02-24
  • 2023-03-13
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多