使用getenv初始化字符串的线程安全问题及解决方案问询

场景与问题 #

项目采用严格的clang-tidy静态分析（版本14.0.6），基于C++17开发，需要获取环境变量值，代码如下：

int main() {
    char const* ompi_size_cstr = std::getenv("OMPI_COMM_WORLD_SIZE");
    ...
}

触发如下警告（已开启warnings-as-errors）：

/builds/user/mpi3/environment.hpp:49:51: error: function is not thread safe [concurrency-mt-unsafe,-warnings-as-errors]
        const char* ompi_size_cstr = std::getenv("OMPI_COMM_WORLD_SIZE");

查阅资料可知，std::getenv自C++11起即为线程安全函数，但警告的本质是：getenv返回的指针指向全局环境变量存储区，该区域可能被其他线程通过setenv/putenv等函数修改，存在数据竞争风险。

希望通过代码优化解决警告，而非直接添加NOLINT忽略规则。

可行解决方案 #

1. 立即拷贝到std::string（通用推荐方案） #

直接将getenv返回的内容拷贝到std::string中，脱离对全局存储区的依赖，彻底消除数据竞争风险：

std::string ompi_size_str = std::getenv("OMPI_COMM_WORLD_SIZE");

若需要const char*，可使用ompi_size_str.c_str()，注意该指针仅在string对象生命周期内有效。

2. 静态初始化std::string（适合启动阶段确定值的场景） #

如果环境变量值在程序启动后不会变更，可将std::string声明为static，利用静态变量的单线程初始化特性，确保拷贝的值稳定：

static const std::string ompi_size_str = std::getenv("OMPI_COMM_WORLD_SIZE");

⚠️ 不推荐使用static const char*：该指针仍指向全局存储区，后续若环境变量被其他线程修改，指针指向的内容会同步变化，仍存在风险。

3. 互斥锁保护访问（需实时获取最新值的场景） #

如果需要每次获取环境变量的最新值，需用互斥锁包裹所有getenv调用及环境变量修改操作：

#include <mutex>

std::mutex env_access_mutex;

// 线程安全的环境变量获取函数
std::string get_env_safe(const char* env_name) {
    std::lock_guard<std::mutex> lock(env_access_mutex);
    const char* raw_val = std::getenv(env_name);
    return raw_val ? std::string(raw_val) : std::string();
}

// 使用示例
std::string ompi_size_str = get_env_safe("OMPI_COMM_WORLD_SIZE");

注意：所有修改环境变量的操作（如setenv）也必须用同一个互斥锁保护，否则仍会存在数据竞争。

4. 最后选择：添加NOLINT注释 #

若确定当前场景下环境变量不会被其他线程修改，可添加带说明的NOLINT注释，避免无意义的忽略：

// NOLINTNEXTLINE(concurrency-mt-unsafe) OMPI_COMM_WORLD_SIZE在程序生命周期内不会被修改
const char* ompi_size_cstr = std::getenv("OMPI_COMM_WORLD_SIZE");

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