]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
projects / ceph.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: c1cd465) | patch
mgr/DaemonServer: handle_conf_change - fix broken locking 29197/head
authorxie xingguo <xie.xingguo@zte.com.cn>
Tue, 26 Mar 2019 08:08:43 +0000 (16:08 +0800)
committerNathan Cutler <ncutler@suse.com>
Tue, 23 Jul 2019 12:20:03 +0000 (14:20 +0200)
commitafacc162b800de87e011673affb90379a05f7edd
tree74a4331dcee4ad59026cd60f1eb6aec55c67be15tree | snapshot
parentc1cd465c4f7a185c193fc8999a74b9ff428ac254commit | diff
mgr/DaemonServer: handle_conf_change - fix broken locking

The is_locked_by_me() is only meant to be safe if you know
you hold the lock. It's meant to be used in an assertion.

Fixes: http://tracker.ceph.com/issues/38899
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
(cherry picked from commit a929a5a01508e661fc7f3cea52822320d39983d2)

Conflicts:
    src/mgr/DaemonServer.cc
- use Mutex::Locker instead of std::lock_guard
src/mgr/DaemonServer.cc diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.