git.apps.os.sepia.ceph.com Git

]> git.apps.os.sepia.ceph.com Git - ceph.git/commit

author	Kefu Chai <kchai@redhat.com>
	Thu, 13 Jul 2017 06:49:48 +0000 (14:49 +0800)
committer	John Spray <john.spray@redhat.com>
	Wed, 1 Nov 2017 23:03:26 +0000 (23:03 +0000)
commit	1eee973b90418979c33ca42b0720045f2de670c3
tree	d6db12f62faea921566dbfa50d2fcf02afe94701	tree \| snapshot
parent	66aa481be983fdd60beddc7ce621dad3edbf5490	commit \| diff

common,mds,mgr,mon,osd: store event only if it's added

otherwise
* we will try to cancel it even it's never been added
* we will keep a dangling pointer around. which is, well,
scaring.
* static analyzer will yell at us:
Memory - illegal accesses (USE_AFTER_FREE)

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 2449b3a5c365987746ada095fde30e3dc63ee0c7)

28 files changed:

src/client/Client.cc		diff \| blob \| history
src/client/Client.h		diff \| blob \| history
src/common/Timer.cc		diff \| blob \| history
src/common/Timer.h		diff \| blob \| history
src/journal/JournalMetadata.cc		diff \| blob \| history
src/journal/ObjectPlayer.cc		diff \| blob \| history
src/journal/ObjectPlayer.h		diff \| blob \| history
src/journal/ObjectRecorder.cc		diff \| blob \| history
src/journal/ObjectRecorder.h		diff \| blob \| history
src/mds/Beacon.cc		diff \| blob \| history
src/mds/Beacon.h		diff \| blob \| history
src/mds/MDSDaemon.cc		diff \| blob \| history
src/mds/MDSDaemon.h		diff \| blob \| history
src/mgr/MgrClient.cc		diff \| blob \| history
src/mon/Elector.cc		diff \| blob \| history
src/mon/MgrMonitor.cc		diff \| blob \| history
src/mon/Monitor.cc		diff \| blob \| history
src/mon/Paxos.cc		diff \| blob \| history
src/mon/PaxosService.cc		diff \| blob \| history
src/osd/PrimaryLogPG.h		diff \| blob \| history
src/osd/Watch.cc		diff \| blob \| history
src/test/perf_local.cc		diff \| blob \| history
src/test/rbd_mirror/mock/MockSafeTimer.h		diff \| blob \| history
src/test/rbd_mirror/test_mock_ImageReplayer.cc		diff \| blob \| history
src/test/rbd_mirror/test_mock_InstanceReplayer.cc		diff \| blob \| history
src/test/rbd_mirror/test_mock_PoolWatcher.cc		diff \| blob \| history
src/tools/rbd_mirror/PoolWatcher.cc		diff \| blob \| history
src/tools/rbd_mirror/image_sync/ImageCopyRequest.cc		diff \| blob \| history

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom