C++的双重检查锁并不安全(转)

一个典型的单例模式构建对象的双重检查锁如下:

复制代码
 1 static Singleton * getSingleObject()
 2     {
 3         if(singleObject==NULL)
 4         {
 5             lock();
 6             if(singleObject==NULL)
 7             {
 8                 singleObject = new Singleton();
 9             }
10             unlock();
11         }
12         return singleObject;
13     }
复制代码

该代码的逻辑是:getSingleObject()函数获得对象,如果对象不存在则创建,反之则直接返回。考虑到线程安全,创建对象时需要加锁(否则可能多线程同时调用该函数时创建多个对象)。同时为了避免线程每次调用getSingleObject()函数时候都要被锁住等待,减小线程间的开销,使用双重检查锁,即只有对象不存在时才锁住等待。

 

看起来天衣无缝……

 

C++在new一个对象时,首先会把指针指向分配的那块空间,然后在初始化该空间。

这样假设线程A执行到new该对象时,分配了指针,但是还未来得及构建那块空间。此时线程B执行getSingleObject()函数,由于指针已经分配所以直接返回该地址(但其实是一块还未初始化的地址),线程B操作该地址,game over。。

 

一种解决方法,但还是有可能出错:

复制代码
 1 class Singleton {
 2 public:
 3 static volatile Singleton* volatile instance();
 4 ...
 5 private:
 6 // one more volatile added 加入另一个volatile声明
 7 static volatile Singleton* volatile pInstance;
 8 };
 9 // from the implementation file 实现文件内容如下
10 volatile Singleton* volatile Singleton::pInstance = 0;
11 volatile Singleton* volatile Singleton::instance() {
12 if (pInstance == 0) {
13 Lock lock;
14 if (pInstance == 0) {
15 // one more volatile added 加入另一个volatile声明
16 volatile Singleton* volatile temp =
17 new volatile Singleton;
18 pInstance = temp;
19 }
20 }
21 return pInstance;
22 }
复制代码

 

posted @   鸭子船长  阅读(1157)  评论(0编辑  收藏  举报
编辑推荐:
· 记一次.NET内存居高不下排查解决与启示
· 探究高空视频全景AR技术的实现原理
· 理解Rust引用及其生命周期标识(上)
· 浏览器原生「磁吸」效果!Anchor Positioning 锚点定位神器解析
· 没有源码,如何修改代码逻辑?
阅读排行:
· 全程不用写代码,我用AI程序员写了一个飞机大战
· MongoDB 8.0这个新功能碉堡了,比商业数据库还牛
· 记一次.NET内存居高不下排查解决与启示
· DeepSeek 开源周回顾「GitHub 热点速览」
· 白话解读 Dapr 1.15:你的「微服务管家」又秀新绝活了
点击右上角即可分享
微信分享提示