C++寄存器接口赋值派生类致未定义行为的修复与禁用方案问询

问题描述 #

编写SPI芯片通信的底层抽象时，通过寄存器结构体替代复杂位操作，定义了IRegister接口类提供asU16方法将结构体转换为uint16_t。直接调用派生类对象的asU16方法正常，但将派生类对象赋值给IRegister对象后调用该方法出现未定义行为——本质是对象切片导致基类对象无法容纳派生类的位域数据，bit_cast访问越界内存。

复现代码：

#include <cstdio>
#include <cstdint>

// interface struct
struct IRegister {
    [[nodiscard]] constexpr uint16_t asU16() {
        return *std::bit_cast<uint16_t*>(this);
    }
};

//Register struct - i have like 20 of those, thats why i used interface 
struct FaultsStatusRegister : IRegister {
    uint16_t CS_OCP_FLT_PHASE_A : 1;
    uint16_t CS_OCP_FLT_PHASE_B : 1;
    uint16_t CS_OCP_FLT_PHASE_C : 1; 

    uint16_t CP_FLT             : 1; 
    uint16_t DVDD_OCP_FLT       : 1; 
    uint16_t DVDD_UV_FLT        : 1; 
    uint16_t DVDD_OV_FLT        : 1; 
    uint16_t BK_OCP_FLT         : 1;
    uint16_t OTS_FLT            : 1;
    uint16_t OTW_FLT            : 1; 
    uint16_t LOCK_FLT           : 1; 
    uint16_t WD_FLT             : 1; 
    uint16_t OTP_FLT            : 1;

    uint16_t Reserved           : 3; 
};

int main()
{
    FaultsStatusRegister reg;
    reg.CS_OCP_FLT_PHASE_C = 1;
    reg.CS_OCP_FLT_PHASE_A = 1;
    reg.CS_OCP_FLT_PHASE_B = 1;
    
    reg.OTP_FLT = 1;    
    
    printf("%b \n", reg.asU16()); //This if fine: 1000000000111
    IRegister ireg = reg;

    printf("%b \n", ireg.asU16()); // UB? : 11100000000

    return 0;
}

解决方案 #

方案1：用CRTP实现无多态的接口复用 #

利用奇异递归模板模式（CRTP）在编译期绑定派生类类型，既复用asU16方法，又从根源避免对象切片：

#include <cstdio>
#include <cstdint>
#include <bit>

template<typename Derived>
struct RegisterBase {
    [[nodiscard]] constexpr uint16_t asU16() const {
        // 强制转换为派生类指针，确保访问完整的寄存器内存
        return *std::bit_cast<const uint16_t*>(static_cast<const Derived*>(this));
    }
};

struct FaultsStatusRegister : RegisterBase<FaultsStatusRegister> {
    uint16_t CS_OCP_FLT_PHASE_A : 1;
    uint16_t CS_OCP_FLT_PHASE_B : 1;
    uint16_t CS_OCP_FLT_PHASE_C : 1; 

    uint16_t CP_FLT             : 1; 
    uint16_t DVDD_OCP_FLT       : 1; 
    uint16_t DVDD_UV_FLT        : 1; 
    uint16_t DVDD_OV_FLT        : 1; 
    uint16_t BK_OCP_FLT         : 1;
    uint16_t OTS_FLT            : 1;
    uint16_t OTW_FLT            : 1; 
    uint16_t LOCK_FLT           : 1; 
    uint16_t WD_FLT             : 1; 
    uint16_t OTP_FLT            : 1;

    uint16_t Reserved           : 3; 
};

int main()
{
    FaultsStatusRegister reg;
    reg.CS_OCP_FLT_PHASE_C = 1;
    reg.CS_OCP_FLT_PHASE_A = 1;
    reg.CS_OCP_FLT_PHASE_B = 1;
    
    reg.OTP_FLT = 1;    
    
    printf("%b \n", reg.asU16()); // 正常输出预期值
    
    // 若尝试定义RegisterBase<FaultsStatusRegister> ireg = reg;
    // 仍会触发切片，但可通过下文方案2彻底禁止
    
    return 0;
}

方案2：编译期禁止基类的不良用法 #

通过删除基类的构造、拷贝构造和赋值运算符，直接禁止将派生类赋值给基类对象的行为，从编译阶段拦截错误：

#include <cstdio>
#include <cstdint>
#include <bit>

struct IRegister {
    // 禁止基类实例化、拷贝和赋值
    IRegister() = delete;
    IRegister(const IRegister&) = delete;
    IRegister& operator=(const IRegister&) = delete;
    
    [[nodiscard]] constexpr uint16_t asU16() const {
        return *std::bit_cast<const uint16_t*>(this);
    }
};

struct FaultsStatusRegister : IRegister {
    // 显式提供派生类的默认构造函数
    FaultsStatusRegister() = default;
    
    uint16_t CS_OCP_FLT_PHASE_A : 1;
    uint16_t CS_OCP_FLT_PHASE_B : 1;
    uint16_t CS_OCP_FLT_PHASE_C : 1; 

    uint16_t CP_FLT             : 1; 
    uint16_t DVDD_OCP_FLT       : 1; 
    uint16_t DVDD_UV_FLT        : 1; 
    uint16_t DVDD_OV_FLT        : 1; 
    uint16_t BK_OCP_FLT         : 1;
    uint16_t OTS_FLT            : 1;
    uint16_t OTW_FLT            : 1; 
    uint16_t LOCK_FLT           : 1; 
    uint16_t WD_FLT             : 1; 
    uint16_t OTP_FLT            : 1;

    uint16_t Reserved           : 3; 
};

int main()
{
    FaultsStatusRegister reg;
    reg.CS_OCP_FLT_PHASE_C = 1;
    reg.CS_OCP_FLT_PHASE_A = 1;
    reg.CS_OCP_FLT_PHASE_B = 1;
    
    reg.OTP_FLT = 1;    
    
    printf("%b \n", reg.asU16()); // 正常输出
    
    // IRegister ireg = reg; // 编译报错，直接禁止该操作
    
    return 0;
}

额外验证建议 #

为确保所有寄存器结构体大小严格为16位，添加静态断言：

static_assert(sizeof(FaultsStatusRegister) == sizeof(uint16_t), "Register must occupy exactly 16 bits");

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