Zerocopy项目中的`addr_of!`安全使用审计分析

在Rust语言中，addr_of!宏用于获取字段或变量的原始指针，而不需要先创建引用。近期Rust语言对addr_of!宏的行为进行了调整（PR #129653），这促使Zerocopy项目团队对其代码库中所有使用addr_of!的地方进行了安全审计。

审计背景

Zerocopy是一个专注于零拷贝操作和内存安全抽象的Rust库。在最新审计中，发现项目中有两处关键位置使用了addr_of!宏：

1. Unalign::get_ptr方法
2. trailing_field_offset!宏

这些使用场景需要重新评估其安全性，特别是考虑到addr_of!现在有更严格的语义要求。

关键问题分析

Unalign::get_ptr方法

该方法通过addr_of!获取内部数据的原始指针。潜在的安全隐患在于：

pub fn get_ptr(&self) -> *const T {
    // 使用addr_of!获取内部值的指针
    ptr::addr_of!(self.value)
}

如果用户获取这个*const T指针后，通过不安全代码将其转换为可变指针并进行修改，就可能违反Rust的内存安全保证。虽然*const T本身是不可变指针，但在不安全代码中转换指针类型是常见做法。

解决方案是在方法文档中明确说明：

• 返回的指针仅用于读取操作
• 任何通过该指针进行的修改行为都是未定义行为
• 用户必须保证不将指针转换为可变指针进行写入

trailing_field_offset!宏

这个宏用于计算结构体尾部字段的偏移量：

let offset = unsafe {
    // 使用addr_of!获取字段地址
    (addr_of!((*ptr).field) as usize).wrapping_sub(ptr as usize)
};

这个使用场景相对安全，因为：

1. 仅用于计算偏移量，不保留指针
2. 不涉及任何实际的内存访问
3. 计算结果仅用于后续的内存布局计算

安全实践建议

基于这次审计，可以总结出在Rust中使用addr_of!的一些最佳实践：

1. 明确指针用途：如果返回的指针仅用于读取，应在文档中明确说明
2. 限制指针生命周期：避免长期持有通过addr_of!获取的指针
3. 避免指针转换：在安全代码中阻止将不可变指针转换为可变指针
4. 充分文档说明：对所有不安全操作提供详尽的文档和安全约定

结论

Zerocopy项目通过这次审计确保了其addr_of!使用场景的安全性。特别是对于Unalign::get_ptr方法，增加了明确的文档约束来防止潜在的内存安全问题。这体现了Rust生态系统对内存安全的高度重视，以及开源项目维护者对安全问题的积极响应。

对于使用Zerocopy库的开发者，建议仔细阅读相关方法的文档说明，确保在使用这些涉及不安全代码的API时遵守所有安全约定。同时，这也提醒所有Rust开发者在类似场景下需要格外注意指针的安全使用。