栈分配对象的容器成员修改触发double free错误，求栈分配修复方案

先直接说核心问题：你的代码里有两个关键错误，其中一个直接触发了double free的内存错误，另一个是不符合右值引用使用意图的不良实践。

1. 直接导致崩溃的根源：Device::setName的返回值错误 #

你定义的setName函数声明返回string，但函数体里没有任何return语句：

string setName(string newName){ m_name = newName; }

这属于未定义行为——程序会从栈上读取随机垃圾数据作为返回值，而string的析构函数会尝试释放这块随机内存，直接引发内存损坏（比如你看到的double free）。

2. 次要但需修正的问题：Server::addDevice未正确处理右值引用 #

虽然这个问题不是崩溃的直接原因，但不符合你使用右值引用的初衷：

void addDevice(Device && dev) { devices.push_back(dev); }

右值引用参数dev在函数内部是左值（有名字的引用都是左值），直接调用push_back(dev)会执行拷贝构造，而不是移动构造，白白做了不必要的内存拷贝。

修复步骤（保持栈分配不变） #

步骤1：修正setName函数 #

把返回类型改成void（setter函数通常不需要返回值），如果确实需要返回修改后的名字，就补充return语句：

// 推荐写法：无返回值的setter
void setName(string newName){ m_name = newName; }

// 可选写法：返回修改后的名字
string setName(string newName){ 
    m_name = newName; 
    return m_name; 
}

步骤2：正确处理右值引用的移动语义 #

在push_back时用std::move把左值化的右值引用转回右值，触发移动构造：

#include <utility> // 需要包含这个头文件才能使用std::move

void addDevice(Device && dev) { 
    devices.push_back(std::move(dev)); 
}

如果你想让addDevice同时支持左值和右值传入，还可以简化成值参数+移动的写法，更灵活：

void addDevice(Device dev) { 
    devices.push_back(std::move(dev)); 
}

修复后的完整代码示例 #

#include <iostream>
#include <vector>
#include <string>
#include <utility>

class Device{
public:
    Device(std::string name):m_name(name) {}
    std::string getName(){ return m_name; }
    void setName(std::string newName){ m_name = newName; } // 修正返回值
private:
    std::string m_name;
};

class Server {
public:
    Device & getDevice(int i) { return devices.at(i); }
    void addDevice(Device && dev) { devices.push_back(std::move(dev)); } // 正确移动右值
private:
    std::vector<Device> devices;
};

int main() {
    Server s{};
    s.addDevice(Device{"ONE"});
    s.addDevice(Device{"TWO"});
    s.addDevice(Device{"THREE"});
    std::cout<<s.getDevice(0).getName()<<std::endl;
    s.getDevice(0).setName("XXX");
    std::cout<<s.getDevice(0).getName()<<std::endl;
    return 0;
}

运行后会正常输出：

ONE
XXX

关于你推测的“释放栈变量”的误解 #

vector里存储的是Device对象的独立实例（拷贝或移动后的），栈上的临时Device对象在addDevice调用结束后会正常销毁，和vector里的对象互不干扰。崩溃的真正原因是setName函数的未定义行为破坏了内存结构，和栈变量的释放无关。

内容的提问来源于stack exchange，提问作者user3070144